Removed an unnecessary NULL check.
I have checked the only place this function is called from.

Signed-off-by: Rickard Strandqvist <rickard_strandqv...@spectrumdigital.se>
---
 drivers/staging/lustre/lustre/mdc/mdc_lib.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/lustre/lustre/mdc/mdc_lib.c 
b/drivers/staging/lustre/lustre/mdc/mdc_lib.c
index e8732cc..a798c2b 100644
--- a/drivers/staging/lustre/lustre/mdc/mdc_lib.c
+++ b/drivers/staging/lustre/lustre/mdc/mdc_lib.c
@@ -224,10 +224,9 @@ void mdc_open_pack(struct ptlrpc_request *req, struct 
md_op_data *op_data,
        rec->cr_fsuid    = from_kuid(&init_user_ns, current_fsuid());
        rec->cr_fsgid    = from_kgid(&init_user_ns, current_fsgid());
        rec->cr_cap      = cfs_curproc_cap_pack();
-       if (op_data != NULL) {
-               rec->cr_fid1 = op_data->op_fid1;
-               rec->cr_fid2 = op_data->op_fid2;
-       }
+       rec->cr_fid1 = op_data->op_fid1;
+       rec->cr_fid2 = op_data->op_fid2;
+
        rec->cr_mode     = mode;
        cr_flags = mds_pack_open_flags(flags, mode);
        rec->cr_rdev     = rdev;
-- 
1.7.10.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to