* Sachin P. Sant [2009-01-15 11:02]:
>
> Dave Anderson wrote:
> > ----- "Bernhard Walle" <bwalle@xxxxxxx> wrote:
> >
> >
> >> * Executing crash without any parameter results in a segmentation fault.
> >> * Add a NULL check for pc->orig_namelist to avoid the segmentation fault.
> >>
> >
> > Huh?
> >
> > The crash utility's program_context structure doesn't even
> > have an "orig_namelist" member.
> >
> > Is this perhaps a bug in the SUSE crash utility derivative?
> >
> >
> Yup not a problem with upstream crash.
Yes, my fault. I was too fast in sending not upstreamed patches
upstream. :-|
Explanation: I have some patch to read gzip'd kernels (mainly for
ia64 /boot/vmlinuz and for x86 /boot/vmlinux.gz). That patch added the
orig_namelist member. Because I extract the /boot/vmlinux.gz to a
temporary file but need the orig_namelist to print out nice error
messages.
I always wanted to 'polish' that patch for upstream inclusion. Any
interest in that feature? Then I would have some motivation to finally
do. :-)
I've chosen that temporary uncompression in favor to replace all reads
and writes with zlib equivalents because it's just extremely less
intrusive.
Bernhard
--
Bernhard Walle, SUSE LINUX Products GmbH, Architecture Development
"I have always wished that my computer would be as easy to use as my
telephone. My wish has come true. I no longer know how to use my
telephone." -- Bjarne Stroustrup
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility
