diff options
author | Rob Herring <robh@kernel.org> | 2018-11-27 08:16:30 -0600 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2018-11-28 09:35:13 +1100 |
commit | f267e674d1452d381bb2f180695801454d3549ad (patch) | |
tree | 3a6fd97fc9fb3aff8e0f68c0d4e8b8d6843369e5 /checks.c | |
parent | 3616b9a811b66744b4b818c2930462b480ddae44 (diff) | |
download | dtc-f267e674d1452d381bb2f180695801454d3549ad.zip dtc-f267e674d1452d381bb2f180695801454d3549ad.tar.gz dtc-f267e674d1452d381bb2f180695801454d3549ad.tar.bz2 |
checks: Fix crash with multiple source annotations
Commit 3616b9a811b6 ("checks: Use source position information for check
failures") causes crashes when there's a check message with multiple
source annotations. Drop the errant addition to the str pointer left
over from the previous version.
Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'checks.c')
-rw-r--r-- | checks.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -123,7 +123,7 @@ static inline void PRINTF(5, 6) check_msg(struct check *c, struct dt_info *dti, pos = pos->next; file_str = srcpos_string(pos); - str += xasprintf_append(&str, " also defined at %s\n", file_str); + xasprintf_append(&str, " also defined at %s\n", file_str); free(file_str); } } |