Re: [PATCH] multipathd: fix issue in 'map $map getprstatus' reply

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH] multipathd: fix issue in 'map $map getprstatus' reply
From: Bart Van Assche <bart.vanassche@xxxxxxxxxxx>
Date: Wed, 3 Aug 2016 09:26:16 -0700
Cc: zhang.kai16@xxxxxxxxxx, dm-devel@xxxxxxxxxx
In-reply-to: <1470208155-6544-1-git-send-email-peng.liang5@zte.com.cn>
References: <1470208155-6544-1-git-send-email-peng.liang5@zte.com.cn>
Spamdiagnosticmetadata: NSPM
Spamdiagnosticoutput: 1:99
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2

On 08/03/2016 12:09 AM, peng.liang5@xxxxxxxxxx wrote:

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


Hello Peng,

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


Thanks,

Bart.

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

Follow-Ups:
- 答复: Re: [PATCH] multipathd: fix issue in 'map $map getprstatus' reply
  - From: peng . liang5

References:
- [PATCH] multipathd: fix issue in 'map $map getprstatus' reply
  - From: peng . liang5

Prev by Date: Re: dm-mq and end_clone_request()
Next by Date: Re: dm-mq and end_clone_request()
Previous by thread: [PATCH] multipathd: fix issue in 'map $map getprstatus' reply
Next by thread: 答复: Re: [PATCH] multipathd: fix issue in 'map $map getprstatus' reply
Index(es):
- Date
- Thread

[Index of Archives] [DM Crypt] [Fedora Desktop] [ATA RAID] [Fedora Marketing] [Fedora Packaging] [Fedora SELinux] [Yosemite Discussion] [KDE Users] [Fedora Docs]

Powered by Linux