[dm-devel] tiny patch: avoid an unnecessary test

[AJ Lewis]

Actually, there's a chance you'd get a NULL pointer error if you don't have
that check if I'm reading the context right.  Linux doesn't guarantee malloc
will return NULL if memory is unavailable, you only get the error when you
attempt to write to memory.  So if there is an allocation error, you're only
going to see it in the memset.

--
AJ

On Fri, Mar 16, 2007 at 09:28:43AM +0100, Jim Meyering wrote:
> Not a big deal (barely worth mentioning), but it might save
> someone else a little time wondering "why".
> 
> 2007-03-11  Jim Meyering  <jim at meyering.net>
> 
> 	Remove unnecessary test.
> 	* dmeventd/dmeventd.c (_alloc_dso_data): Don't test known-non-NULL
> 	memset return value.
> 
> 2007-03-11  Jim Meyering  <jim at meyering.net>
> Index: dmeventd/dmeventd.c
> ===================================================================
> RCS file: /cvs/dm/device-mapper/dmeventd/dmeventd.c,v
> retrieving revision 1.46
> diff -u -p -r1.46 dmeventd.c
> --- dmeventd/dmeventd.c	2 Feb 2007 17:08:51 -0000	1.46
> +++ dmeventd/dmeventd.c	12 Mar 2007 14:39:35 -0000
> @@ -258,8 +258,8 @@ static struct dso_data *_alloc_dso_data(
>  	if (!ret)
>  		return NULL;
> 
> -	if (!memset(ret, 0, sizeof(*ret)) ||
> -	    !(ret->dso_name = dm_strdup(data->dso_name))) {
> +	memset(ret, 0, sizeof(*ret));
> +	if (!(ret->dso_name = dm_strdup(data->dso_name))) {
>  		dm_free(ret);
>  		return NULL;
>  	}
> 
> --
> dm-devel mailing list
> dm-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 186 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20070316/59a5bfa3/attachment.sig>