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, [email protected] 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.

--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to