> > The problem is a NULL pointer is being passed to _SCI_STRDUP, which then > > passes it to strdup(). For me, it was doing it on the options for the gfx > > driver. I emailed Alex (who made the change that introduced the problem) > > earlier today, hopefully he'll look into a proper fix tonight. > > Oops, that's probably my mistake. I changed the original > > gfx_driver_name = cl_options.gfx_driver_name; > free(cl_options.gfx_driver_name); > > (which, of course, destroys the gfx driver name string) to > > gfx_driver_name = sci_strdup(cl_options.gfx_driver_name); > free(cl_options.gfx_driver_name); > > not realising that cl_options.gfx_driver_name might be NULL. > Maybe it should be changed to the original without the free?
Yes, it _wasn't_ me! I'm about to commit a fix. Alex.
