diff options
author | Jeremy Sowden <jeremy@azazel.net> | 2022-12-03 19:02:11 +0000 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2022-12-08 21:55:21 +0100 |
commit | 2ae7a3f96563c6cf3ea21445e7958314b6ca20fb (patch) | |
tree | fd7161d47ccad2d711985d7ff542fa0dcdc81196 /output/ulogd_output_JSON.c | |
parent | 49f6def6fcbaf01f395fbe00543a9ab2c4bb106e (diff) |
JSON: remove incorrect config value check
The `u.string` member of a config entry is an array, and so never `NULL`.
Output the device string unconditionally.
Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'output/ulogd_output_JSON.c')
-rw-r--r-- | output/ulogd_output_JSON.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/output/ulogd_output_JSON.c b/output/ulogd_output_JSON.c index bbc3dba..798cb48 100644 --- a/output/ulogd_output_JSON.c +++ b/output/ulogd_output_JSON.c @@ -276,8 +276,8 @@ static int json_interp_file(struct ulogd_pluginstance *upi, char *buf) static int json_interp(struct ulogd_pluginstance *upi) { struct json_priv *opi = (struct json_priv *) &upi->private; + char *dvc, *buf, *tmp; unsigned int i; - char *buf, *tmp; size_t buflen; json_t *msg; @@ -335,10 +335,8 @@ static int json_interp(struct ulogd_pluginstance *upi) json_object_set_new(msg, "timestamp", json_string(timestr)); } - if (upi->config_kset->ces[JSON_CONF_DEVICE].u.string) { - char *dvc = upi->config_kset->ces[JSON_CONF_DEVICE].u.string; - json_object_set_new(msg, "dvc", json_string(dvc)); - } + dvc = upi->config_kset->ces[JSON_CONF_DEVICE].u.string; + json_object_set_new(msg, "dvc", json_string(dvc)); for (i = 0; i < upi->input.num_keys; i++) { struct ulogd_key *key = upi->input.keys[i].u.source; |