[PATCH] libdwfl: Correct nested asprintf result check in report_kernel_archive.

Frank Ch. Eigler fche at redhat.com
Sat Dec 21 17:46:19 UTC 2013


Hi -

> Yeah, I think you are right, I overdid it because I was a little embarrassed
> I had to count the brackets 6 times before seeing the bug even after it was
> pointed out to me. Pushed the simplified version:

Aw heck, at that point, why not

...

int res = asprintf (&archive,
                    ((*release)[0]=='/' ? "%s/debug.a"
                                        : MODULEDIRFMT "/debug.a"),
                    *release);

- FChE


More information about the elfutils-devel mailing list