[Crash-utility] [PATCH] Fix segmentation fault

[Bernhard Walle]

* Sachin P. Sant [2009-01-15 11:02]:
>
> Dave Anderson wrote:
> > ----- "Bernhard Walle" <bwalle at suse.de> 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