From: Mark Wielaard Date: Tue, 5 Jun 2018 21:34:46 +0000 (+0200) Subject: libdw: Explicitly check we could decode diridx in dwarf_getsrclines. X-Git-Tag: elfutils-0.172~17 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7ae7ec1ad75dd0689b5371c2e157165c4c7944c9;p=platform%2Fupstream%2Felfutils.git libdw: Explicitly check we could decode diridx in dwarf_getsrclines. It is highly unlikely dwarf_formudata fails because we setup the attribute ourselves, but better to explicitly mark diridx as bad if it does. Signed-off-by: Mark Wielaard --- diff --git a/libdw/ChangeLog b/libdw/ChangeLog index 661fa9d..21adeb7 100644 --- a/libdw/ChangeLog +++ b/libdw/ChangeLog @@ -1,5 +1,10 @@ 2018-06-05 Mark Wielaard + * dwarf_getsrclines.c (read_srclines): Explicitly set diridx to -1 + in case dwarf_formudata fails. + +2018-06-05 Mark Wielaard + * dwarf_getaranges (dwarf_getaranges): Free new_arange if __libdw_findcu fails. diff --git a/libdw/dwarf_getsrclines.c b/libdw/dwarf_getsrclines.c index 0c2efaa..07baebc 100644 --- a/libdw/dwarf_getsrclines.c +++ b/libdw/dwarf_getsrclines.c @@ -566,7 +566,7 @@ read_srclines (Dwarf *dbg, for (unsigned int n = 0; n < nfiles; n++) { const char *fname = NULL; - Dwarf_Word diridx = -1; + Dwarf_Word diridx = (Dwarf_Word) -1; for (unsigned char m = 0; m < nforms; m++) { if (m == form_path) @@ -581,7 +581,8 @@ read_srclines (Dwarf *dbg, attr.code = DW_AT_decl_file; /* Close enough. */ attr.form = forms[m]; attr.valp = (void *) linep; - dwarf_formudata (&attr, &diridx); + if (dwarf_formudata (&attr, &diridx) != 0) + diridx = (Dwarf_Word) -1; } size_t len = __libdw_form_val_len (&fake_cu, forms[m], linep);