Seems right. Merged, with thanks. On Tue, Oct 11, 2016 at 8:37 AM, <tang.jun...@zte.com.cn> wrote:
> Please have a review for this patch, any comment will be highly > appreciated. > > > > > 发件人: tang.jun...@zte.com.cn > 收件人: christophe varoqui <christophe.varo...@free.fr>, > chau...@redhat.com, Vijay <vijay.chau...@netapp.com>, Benjamin Marzinski < > bmarz...@redhat.com>, > 抄送: zhang.ka...@zte.com.cn, dm-devel@redhat.com, "tang.junhui" < > tang.jun...@zte.com.cn> > 日期: 2016/09/21 16:54 > 主题: [dm-devel] [PATCH] mpathpersist: memset length is wrong > 发件人: dm-devel-boun...@redhat.com > ------------------------------ > > > > From: "tang.junhui" <tang.jun...@zte.com.cn> > > variable transportids is cleared by memset() with wrong length > MPATH_MX_TIDS, > the length should be MPATH_MX_TIDS*sizeof(struct transportid). > > Signed-off-by: tang.junhui <tang.jun...@zte.com.cn> > --- > mpathpersist/main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mpathpersist/main.c b/mpathpersist/main.c > index a55865f..8e8cc35 100644 > --- a/mpathpersist/main.c > +++ b/mpathpersist/main.c > @@ -105,7 +105,7 @@ int main (int argc, char * argv[]) > > udev = udev_new(); > conf = mpath_lib_init(udev); > - memset(transportids,0,MPATH_MX_TIDS); > + memset(transportids, 0, MPATH_MX_TIDS * sizeof(struct > transportid)); > multipath_conf = conf; > > while (1) > -- > 2.8.1.windows.1 > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel > >
-- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel