<font size=2>Please have a review</font><font size=2 face="sans-serif">
for this patch, any comment will be highly appreciated.<br>
</font>
<br>
<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">发件人:    
    </font><font size=1 face="sans-serif">tang.junhui@zte.com.cn</font>
<br><font size=1 color=#5f5f5f face="sans-serif">收件人:    
    </font><font size=1 face="sans-serif">christophe
varoqui <christophe.varoqui@free.fr>, Chauhan@redhat.com, Vijay <Vijay.Chauhan@netapp.com>,
Benjamin Marzinski <bmarzins@redhat.com>, </font>
<br><font size=1 color=#5f5f5f face="sans-serif">抄送:    
   </font><font size=1 face="sans-serif">zhang.kai16@zte.com.cn,
dm-devel@redhat.com, "tang.junhui" <tang.junhui@zte.com.cn></font>
<br><font size=1 color=#5f5f5f face="sans-serif">日期:    
    </font><font size=1 face="sans-serif">2016/09/21
16:54</font>
<br><font size=1 color=#5f5f5f face="sans-serif">主题:    
   </font><font size=1 face="sans-serif">[dm-devel] [PATCH]
mpathpersist: memset length is wrong</font>
<br><font size=1 color=#5f5f5f face="sans-serif">发件人:    
   </font><font size=1 face="sans-serif">dm-devel-bounces@redhat.com</font>
<br>
<hr noshade>
<br>
<br>
<br><tt><font size=2>From: "tang.junhui" <tang.junhui@zte.com.cn><br>
<br>
variable transportids is cleared by memset() with wrong length MPATH_MX_TIDS,<br>
the length should be MPATH_MX_TIDS*sizeof(struct transportid).<br>
<br>
Signed-off-by: tang.junhui <tang.junhui@zte.com.cn><br>
---<br>
 mpathpersist/main.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/mpathpersist/main.c b/mpathpersist/main.c<br>
index a55865f..8e8cc35 100644<br>
--- a/mpathpersist/main.c<br>
+++ b/mpathpersist/main.c<br>
@@ -105,7 +105,7 @@ int main (int argc, char * argv[])<br>
 <br>
                  udev
= udev_new();<br>
                  conf
= mpath_lib_init(udev);<br>
-                
memset(transportids,0,MPATH_MX_TIDS);<br>
+                
memset(transportids, 0, MPATH_MX_TIDS * sizeof(struct transportid));<br>
                  multipath_conf
= conf;<br>
 <br>
                  while
(1)<br>
-- <br>
2.8.1.windows.1<br>
<br>
--<br>
dm-devel mailing list<br>
dm-devel@redhat.com<br>
</font></tt><a href="https://www.redhat.com/mailman/listinfo/dm-devel"><tt><font size=2>https://www.redhat.com/mailman/listinfo/dm-devel</font></tt></a><tt><font size=2><br>
</font></tt>
<br>