diff options
| author | Mikhail Gusarov <dottedmag@dottedmag.net> | 2010-06-04 16:58:58 +0700 |
|---|---|---|
| committer | Mikhail Gusarov <dottedmag@dottedmag.net> | 2010-06-11 19:04:23 +0700 |
| commit | 6592db6bb526f0c43b4c7b55859c629709e039b4 (patch) | |
| tree | 30d525b635170a94193c0a8837e1b36b609e757e /config/udev.c | |
| parent | 620ca54aaa0b363fcf68cec1bd6c37e68c988352 (diff) | |
Get rid of xstrdup when argument is definitely non-NULL
Replace xstrdup with strdup when either constant string is
being duplicated or argument is guarded by conditionals and
obviously can't be NULL
Signed-off-by: Mikhail Gusarov <dottedmag@dottedmag.net>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'config/udev.c')
| -rw-r--r-- | config/udev.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/config/udev.c b/config/udev.c index 16c462455..993449075 100644 --- a/config/udev.c +++ b/config/udev.c @@ -86,8 +86,8 @@ device_added(struct udev_device *udev_device) if (!options) return; - options->key = xstrdup("_source"); - options->value = xstrdup("server/udev"); + options->key = strdup("_source"); + options->value = strdup("server/udev"); if (!options->key || !options->value) goto unwind; @@ -197,7 +197,7 @@ device_added(struct udev_device *udev_device) for (; dev; dev = dev->next) { free(dev->config_info); - dev->config_info = xstrdup(config_info); + dev->config_info = strdup(config_info); } unwind: |