[dm-devel] 答复: Re: [PATCH] multipathd: fix issue in 'map $map getprstatus' reply

peng.liang5 at zte.com.cn peng.liang5 at zte.com.cn
Thu Aug 4 01:39:24 UTC 2016


Hi Bart,

Thanks again.
I got it.

-       *len = 3;
-       asprintf(reply, "%d\n", mpp->prflag);
+       *len = asprintf(reply, "%d\n", mpp->prflag);
+       if (*len < 0)
+               return 1;

On 08/03/2016 12:09 AM, peng.liang5 at zte.com.cn wrote:
> +              *len = 3;
> +              asprintf(reply, "%d\n", mpp->prflag);

Hello Peng,

Please use the return value of asprintf() to compute *len instead of 
assigning a hard-coded constant to *len.

Thanks,

Bart.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20160804/568edf1d/attachment.htm>


More information about the dm-devel mailing list