On Sun, 2014-12-21 at 23:08 +0100, Rickard Strandqvist wrote:
> Removes same unnecessary NULL check.
Depending on settings, these checks are necessary:
drivers/staging/lustre/lustre/include/obd_support.h:#define POISON(ptr, c, s)
memset(ptr, c, s)
[]
drivers/staging/lustre/lustre/include/obd_support.h:#define OBD_FREE_PRE(ptr,
size, name) \
drivers/staging/lustre/lustre/include/obd_support.h- LASSERT(ptr);
\
drivers/staging/lustre/lustre/include/obd_support.h- obd_memory_sub(size);
\
drivers/staging/lustre/lustre/include/obd_support.h- CDEBUG(D_MALLOC, name "
'" #ptr "': %d at %p.\n", \
drivers/staging/lustre/lustre/include/obd_support.h- (int)(size),
ptr); \
drivers/staging/lustre/lustre/include/obd_support.h- POISON(ptr, 0x5a, size)
[]
drivers/staging/lustre/lustre/include/obd_support.h:#define OBD_FREE(ptr, size)
\
drivers/staging/lustre/lustre/include/obd_support.h-do {
\
drivers/staging/lustre/lustre/include/obd_support.h- OBD_FREE_PRE(ptr, size,
"kfreed"); \
drivers/staging/lustre/lustre/include/obd_support.h- kfree(ptr);
\
drivers/staging/lustre/lustre/include/obd_support.h- POISON_PTR(ptr);
\
drivers/staging/lustre/lustre/include/obd_support.h-} while (0)
> diff --git a/drivers/staging/lustre/lustre/llite/llite_lib.c
> b/drivers/staging/lustre/lustre/llite/llite_lib.c
[]
> @@ -591,10 +591,8 @@ static int client_common_fill_super(struct super_block
> *sb, char *md, char *dt,
> get_uuid2fsid(uuid->uuid, strlen(uuid->uuid), &sbi->ll_fsid);
> }
>
> - if (data != NULL)
> - OBD_FREE_PTR(data);
> - if (osfs != NULL)
> - OBD_FREE_PTR(osfs);
> + OBD_FREE_PTR(data);
> + OBD_FREE_PTR(osfs);
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
