[PATCH] xfree86: use xnfstrdup in the Xorg -configure code

Hans de Goede hdegoede at redhat.com
Wed Feb 5 22:24:11 PST 2014


Hi,

On 02/05/2014 11:05 PM, Peter Hutterer wrote:
> Just for consistency, I'm pretty sure the code is generally not happy for
> malloc failures anyway.
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> Easier to do it that way given that there are already plenty of strdups in
> the -configure code. This fixes up that bit (included with the recent
> patches), the rest of the server is still a bit random for handling malloc
> failures.
>
>   hw/xfree86/common/xf86Configure.c | 42 +++++++++++++++++++--------------------
>   1 file changed, 21 insertions(+), 21 deletions(-)

Looks good,

Reviewed-by: Hans de Goede <hdegoede at redhat.com>

>
> diff --git a/hw/xfree86/common/xf86Configure.c b/hw/xfree86/common/xf86Configure.c
> index 6aaf634..967bfbc 100644
> --- a/hw/xfree86/common/xf86Configure.c
> +++ b/hw/xfree86/common/xf86Configure.c
> @@ -159,8 +159,8 @@ configureInputSection(void)
>
>       parsePrologue(XF86ConfInputPtr, XF86ConfInputRec)
>
> -    ptr->inp_identifier = strdup("Keyboard0");
> -    ptr->inp_driver = strdup("kbd");
> +    ptr->inp_identifier = xnfstrdup("Keyboard0");
> +    ptr->inp_driver = xnfstrdup("kbd");
>       ptr->list.next = NULL;
>
>       /* Crude mechanism to auto-detect mouse (os dependent) */
> @@ -175,17 +175,17 @@ configureInputSection(void)
>       }
>
>       mouse = calloc(1, sizeof(XF86ConfInputRec));
> -    mouse->inp_identifier = strdup("Mouse0");
> -    mouse->inp_driver = strdup("mouse");
> +    mouse->inp_identifier = xnfstrdup("Mouse0");
> +    mouse->inp_driver = xnfstrdup("mouse");
>       mouse->inp_option_lst =
> -        xf86addNewOption(mouse->inp_option_lst, strdup("Protocol"),
> -                         strdup(DFLT_MOUSE_PROTO));
> +        xf86addNewOption(mouse->inp_option_lst, xnfstrdup("Protocol"),
> +                         xnfstrdup(DFLT_MOUSE_PROTO));
>       mouse->inp_option_lst =
> -        xf86addNewOption(mouse->inp_option_lst, strdup("Device"),
> -                         strdup(DFLT_MOUSE_DEV));
> +        xf86addNewOption(mouse->inp_option_lst, xnfstrdup("Device"),
> +                         xnfstrdup(DFLT_MOUSE_DEV));
>       mouse->inp_option_lst =
> -        xf86addNewOption(mouse->inp_option_lst, strdup("ZAxisMapping"),
> -                         strdup("4 5 6 7"));
> +        xf86addNewOption(mouse->inp_option_lst, xnfstrdup("ZAxisMapping"),
> +                         xnfstrdup("4 5 6 7"));
>       ptr = (XF86ConfInputPtr) xf86addListItem((glp) ptr, (glp) mouse);
>       return ptr;
>   }
> @@ -290,7 +290,7 @@ configureDeviceSection(int screennum)
>               "        ### <string>: \"String\", <freq>: \"<f> Hz/kHz/MHz\",\n"
>               "        ### <percent>: \"<f>%\"\n"
>               "        ### [arg]: arg optional\n";
> -        ptr->dev_comment = strdup(descrip);
> +        ptr->dev_comment = xnfstrdup(descrip);
>           if (ptr->dev_comment) {
>               for (p = DevToConfig[screennum].GDev.options; p->name != NULL; p++) {
>                   char *p_e;
> @@ -337,9 +337,9 @@ configureLayoutSection(void)
>           iptr = malloc(sizeof(XF86ConfInputrefRec));
>           iptr->list.next = NULL;
>           iptr->iref_option_lst = NULL;
> -        iptr->iref_inputdev_str = strdup("Mouse0");
> +        iptr->iref_inputdev_str = xnfstrdup("Mouse0");
>           iptr->iref_option_lst =
> -            xf86addNewOption(iptr->iref_option_lst, strdup("CorePointer"),
> +            xf86addNewOption(iptr->iref_option_lst, xnfstrdup("CorePointer"),
>                                NULL);
>           ptr->lay_input_lst = (XF86ConfInputrefPtr)
>               xf86addListItem((glp) ptr->lay_input_lst, (glp) iptr);
> @@ -351,9 +351,9 @@ configureLayoutSection(void)
>           iptr = malloc(sizeof(XF86ConfInputrefRec));
>           iptr->list.next = NULL;
>           iptr->iref_option_lst = NULL;
> -        iptr->iref_inputdev_str = strdup("Keyboard0");
> +        iptr->iref_inputdev_str = xnfstrdup("Keyboard0");
>           iptr->iref_option_lst =
> -            xf86addNewOption(iptr->iref_option_lst, strdup("CoreKeyboard"),
> +            xf86addNewOption(iptr->iref_option_lst, xnfstrdup("CoreKeyboard"),
>                                NULL);
>           ptr->lay_input_lst = (XF86ConfInputrefPtr)
>               xf86addListItem((glp) ptr->lay_input_lst, (glp) iptr);
> @@ -431,9 +431,9 @@ configureFilesSection(void)
>       parsePrologue(XF86ConfFilesPtr, XF86ConfFilesRec)
>
>           if (xf86ModulePath)
> -        ptr->file_modulepath = strdup(xf86ModulePath);
> +        ptr->file_modulepath = xnfstrdup(xf86ModulePath);
>       if (defaultFontPath)
> -        ptr->file_fontpath = strdup(defaultFontPath);
> +        ptr->file_fontpath = xnfstrdup(defaultFontPath);
>
>       return ptr;
>   }
> @@ -446,8 +446,8 @@ configureMonitorSection(int screennum)
>
>       XNFasprintf(&tmp, "Monitor%d", screennum);
>       ptr->mon_identifier = tmp;
> -    ptr->mon_vendor = strdup("Monitor Vendor");
> -    ptr->mon_modelname = strdup("Monitor Model");
> +    ptr->mon_vendor = xnfstrdup("Monitor Vendor");
> +    ptr->mon_modelname = xnfstrdup("Monitor Model");
>
>       return ptr;
>   }
> @@ -491,7 +491,7 @@ configureDDCMonitorSection(int screennum)
>
>       XNFasprintf(&tmp, "Monitor%d", screennum);
>       ptr->mon_identifier = tmp;
> -    ptr->mon_vendor = strdup(ConfiguredMonitor->vendor.name);
> +    ptr->mon_vendor = xnfstrdup(ConfiguredMonitor->vendor.name);
>       XNFasprintf(&ptr->mon_modelname, "%x", ConfiguredMonitor->vendor.prod_id);
>
>       /* features in centimetres, we want millimetres */
> @@ -529,7 +529,7 @@ configureDDCMonitorSection(int screennum)
>
>       if (ConfiguredMonitor->features.dpms) {
>           ptr->mon_option_lst =
> -            xf86addNewOption(ptr->mon_option_lst, strdup("DPMS"), NULL);
> +            xf86addNewOption(ptr->mon_option_lst, xnfstrdup("DPMS"), NULL);
>       }
>
>       return ptr;
>


Regards,

Hans


More information about the xorg-devel mailing list