[PATCH] return when not finding the name
Marcus Meissner
marcus at jet.franken.de
Sun Feb 18 07:39:21 CST 2007
On Sun, Feb 18, 2007 at 01:40:12PM +0100, Eric Pouech wrote:
> Marcus Meissner a écrit :
> >---
> > dlls/dbghelp/dwarf.c | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> >diff --git a/dlls/dbghelp/dwarf.c b/dlls/dbghelp/dwarf.c
> >index 4f74b6a..f51a948 100644
> >--- a/dlls/dbghelp/dwarf.c
> >+++ b/dlls/dbghelp/dwarf.c
> >@@ -1203,7 +1203,7 @@ static void dwarf2_parse_enumerator(dwar
> >
> > TRACE("%s, for %s\n", dwarf2_debug_ctx(ctx), dwarf2_debug_di(di));
> >
> >- if (!dwarf2_find_attribute(ctx, di, DW_AT_name, &name)) name.u.string
> >= NULL;
> >+ if (!dwarf2_find_attribute(ctx, di, DW_AT_name, &name)) return;
> > if (!dwarf2_find_attribute(ctx, di, DW_AT_const_value, &value))
> > value.u.svalue = 0;
> > symt_add_enum_element(ctx->module, parent, name.u.string,
> > value.u.svalue);
> >
> >
> Hi Marcus,
>
> can you send me the ELF file(s) which is/are causing this?
There is no ELF file.
Coverity spotted this as error, since name.u.string can be NULL
in the flow and symt_add_enum_element() would crash in pool_strdup.
It is CID 461. 462 and 463 are related issues.
Ciao, Marcus
More information about the wine-devel
mailing list