summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJason Nader <jason.nader@protonmail.com>2021-10-25 13:28:32 +0900
committerSimon Ser <contact@emersion.fr>2021-10-25 10:18:40 +0200
commit9303bed4d4523d158b33c44f534a53b21d7688d6 (patch)
tree55965bc247a85cac9c240a1e5715a7a9f6c38ddb
parentb223f702500d7cda6336e948062d680f07f34f66 (diff)
refactor: use JSON_MAX_DEPTH everywhere
-rw-r--r--include/ipc-client.h3
-rw-r--r--swaybar/ipc.c2
-rw-r--r--swaymsg/main.c3
3 files changed, 4 insertions, 4 deletions
diff --git a/include/ipc-client.h b/include/ipc-client.h
index d3895023f..9c5712d73 100644
--- a/include/ipc-client.h
+++ b/include/ipc-client.h
@@ -1,6 +1,9 @@
#ifndef _SWAY_IPC_CLIENT_H
#define _SWAY_IPC_CLIENT_H
+// arbitrary number, it's probably sufficient, higher number = more memory usage
+#define JSON_MAX_DEPTH 512
+
#include <stdbool.h>
#include <stdint.h>
#include <sys/time.h>
diff --git a/swaybar/ipc.c b/swaybar/ipc.c
index a64aa1abf..2cb235bf4 100644
--- a/swaybar/ipc.c
+++ b/swaybar/ipc.c
@@ -550,7 +550,7 @@ bool handle_ipc_readable(struct swaybar *bar) {
// The default depth of 32 is too small to represent some nested layouts, but
// we can't pass INT_MAX here because json-c (as of this writing) prefaults
// all the memory for its stack.
- json_tokener *tok = json_tokener_new_ex(256);
+ json_tokener *tok = json_tokener_new_ex(JSON_MAX_DEPTH);
if (!tok) {
sway_log_errno(SWAY_ERROR, "failed to create tokener");
free_ipc_response(resp);
diff --git a/swaymsg/main.c b/swaymsg/main.c
index e469319a7..0d9dc5a0d 100644
--- a/swaymsg/main.c
+++ b/swaymsg/main.c
@@ -1,8 +1,5 @@
#define _POSIX_C_SOURCE 200809L
-// arbitrary number, it's probably sufficient, higher number = more memory usage
-#define JSON_MAX_DEPTH 512
-
#include <limits.h>
#include <stdio.h>
#include <stdlib.h>