On Wed, Dec 10, 2014 at 10:05:49PM +1100, Jonathan Gray wrote: > On Wed, Dec 10, 2014 at 10:39:37AM +0100, Sébastien Marie wrote: > > > > Hi, > > > > Fuzzing sed with afl, I found a crash due to use of uninitialized > > variable. > > > > In process.c oldpsl variable need to be initialized: > > > > $ echo | sed -e 'g;P' > > Segmentation fault (core dumped) > > > > The following patch correct this. > > > > I also include the initialization of p, as it is reported by compiler > > warning too (with -Wall -O2). > > Perhaps the following instead to rework the code to seperate out > the path that needs the pointer swap? >
ok, no problem with this version. -- Sébastien Marie