Message ID | 0cbfeea7-5f30-a148-555b-19af2429c32c@huawei.com (mailing list archive) |
---|---|
State | Not Applicable, archived |
Delegated to: | christophe varoqui |
Headers | show |
Series | devmapper: remove useless using of memset in dm_get_info func | expand |
On Thu, Jul 30, 2020 at 10:06:35PM +0800, Zhiqiang Liu wrote: > > In dm_get_info func, if do_get_info fails, we donot need to > call memset to assign 0 to *dmi which will be freed. > > Here, we remove the useless using of memset in dm_get_info func. > > Signed-off-by: Zhiqiang Liu <liuzhiqiang26@huawei.com> Reviewed-by: Benjamin Marzinski <bmarzins@redhat.com> > --- > libmultipath/devmapper.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libmultipath/devmapper.c b/libmultipath/devmapper.c > index f597ff8b..00ee0dfc 100644 > --- a/libmultipath/devmapper.c > +++ b/libmultipath/devmapper.c > @@ -1464,7 +1464,6 @@ dm_get_info (const char * mapname, struct dm_info ** dmi) > return 1; > > if (do_get_info(mapname, *dmi) != 0) { > - memset(*dmi, 0, sizeof(struct dm_info)); > FREE(*dmi); > *dmi = NULL; > return 1; > -- > 2.24.0.windows.2 > -- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel
diff --git a/libmultipath/devmapper.c b/libmultipath/devmapper.c index f597ff8b..00ee0dfc 100644 --- a/libmultipath/devmapper.c +++ b/libmultipath/devmapper.c @@ -1464,7 +1464,6 @@ dm_get_info (const char * mapname, struct dm_info ** dmi) return 1; if (do_get_info(mapname, *dmi) != 0) { - memset(*dmi, 0, sizeof(struct dm_info)); FREE(*dmi); *dmi = NULL; return 1;
In dm_get_info func, if do_get_info fails, we donot need to call memset to assign 0 to *dmi which will be freed. Here, we remove the useless using of memset in dm_get_info func. Signed-off-by: Zhiqiang Liu <liuzhiqiang26@huawei.com> --- libmultipath/devmapper.c | 1 - 1 file changed, 1 deletion(-)