<font size=2 face="sans-serif">Hello Bart,</font>
<br>
<br><font size=2 face="sans-serif">Thanks for your review. Using calloc()
is really better than malloc(). I will modify and commit it later.</font>
<br>
<br><font size=2 face="sans-serif">Thanks,</font>
<br>
<br><font size=2 face="sans-serif">Ding Yemin.<br>
</font><font size=3> </font>
<br>
<br>
<br>
<table width=100%>
<tr valign=top>
<td width=36%><font size=1 face="sans-serif"><b>Bart Van Assche <bart.vanassche@sandisk.com></b>
</font>
<p><font size=1 face="sans-serif">2016-10-28 23:16</font>
<td width=63%>
<table width=100%>
<tr valign=top>
<td>
<div align=right><font size=1 face="sans-serif">收件人</font></div>
<td><font size=1 face="sans-serif"><ding.yemin@zte.com.cn>, Christophe
Varoqui <christophe.varoqui@opensvc.com>, </font>
<tr valign=top>
<td>
<div align=right><font size=1 face="sans-serif">抄送</font></div>
<td><font size=1 face="sans-serif"><tang.junhui@zte.com.cn>, <zhang.kai16@zte.com.cn>,
<dm-devel@redhat.com></font>
<tr valign=top>
<td>
<div align=right><font size=1 face="sans-serif">主题</font></div>
<td><font size=1 face="sans-serif">Re: [dm-devel] [PATCH] Remove duplicated
code in mpath_alloc_prin_response()</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><tt><font size=2>On 10/25/2016 01:19 AM, ding.yemin@zte.com.cn wrote:<br>
>                  
}<br>
> +                
if (size > 0)<br>
> +                
{<br>
> +                
                 ptr
= malloc(size);<br>
> +                
                 if
(ptr)<br>
> +                
                 {<br>
> +                
                 
               
memset(ptr, 0, size);<br>
> +                
                 }<br>
> +                
}<br>
>                  
return ptr;<br>
>  }<br>
<br>
Hello Ding Yemin,<br>
<br>
Please use calloc() instead of open coding it.<br>
<br>
Thanks,<br>
<br>
Bart.<br>
<br>
</font></tt>
<br>