Hi Matt!

You wrote:

> 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?

-- 
Kind regards,
+---------------------------------------------------------------+
| Bas Zoetekouw                  | Si l'on sait exactement ce   |
|--------------------------------| que l'on va faire, a quoi    |
| [EMAIL PROTECTED]             | bon le faire?                |
|    [EMAIL PROTECTED]             |               Pablo Picasso  |
+---------------------------------------------------------------+ 

Reply via email to