diff options
author | Luca Boccassi <luca.boccassi@microsoft.com> | 2021-04-07 10:15:20 +0100 |
---|---|---|
committer | Luca Boccassi <luca.boccassi@microsoft.com> | 2021-04-07 14:02:38 +0100 |
commit | 26c22c84e0d99e645057e7c77e36dbdb606c1e27 (patch) | |
tree | 7c5f446699eec527a24783643e9d9a1575dd6e62 /src/coredump | |
parent | 2c173dbf1b31639e07915c2715ba12db531e1497 (diff) | |
download | systemd-26c22c84e0d99e645057e7c77e36dbdb606c1e27.tar.gz |
coredump: use set_put_strdup()
Follow-up for 95f71807733
Diffstat (limited to 'src/coredump')
-rw-r--r-- | src/coredump/stacktrace.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/coredump/stacktrace.c b/src/coredump/stacktrace.c index b37f02b21e..cc31c7b03c 100644 --- a/src/coredump/stacktrace.c +++ b/src/coredump/stacktrace.c @@ -166,7 +166,6 @@ static int parse_package_metadata(const char *name, JsonVariant *id_json, Elf *e * magic ID is always the same. */ if (note_header.n_type == ELF_PACKAGE_METADATA_ID) { _cleanup_(json_variant_unrefp) JsonVariant *v = NULL, *w = NULL; - char *name_key = NULL; r = json_parse(payload, 0, &v, NULL, NULL); if (r < 0) { @@ -206,14 +205,9 @@ static int parse_package_metadata(const char *name, JsonVariant *id_json, Elf *e } /* Finally stash the name, so we avoid double visits. */ - name_key = strdup(name); - if (!name_key) { - log_oom(); - return DWARF_CB_ABORT; - } - r = set_ensure_consume(c->modules, &string_hash_ops, name_key); + r = set_put_strdup(c->modules, name); if (r < 0) { - log_error_errno(r, "set_ensure_consume failed: %m"); + log_error_errno(r, "set_put_strdup failed: %m"); return DWARF_CB_ABORT; } |