Whamcloud - gitweb
LU-10504 flr: check layout pointer before using it 15/30915/2
authorJian Yu <jian.yu@intel.com>
Thu, 18 Jan 2018 06:44:14 +0000 (22:44 -0800)
committerOleg Drokin <oleg.drokin@intel.com>
Thu, 25 Jan 2018 04:48:33 +0000 (04:48 +0000)
This patch fixes mirror_create() to check layout pointer
before using it.

Change-Id: Ia1454b5c7fcfcee227d0b954a477cefe5d7bb5f7
Test-Parameters: trivial testlist=sanity-flr
Signed-off-by: Jian Yu <jian.yu@intel.com>
Reviewed-on: https://review.whamcloud.com/30915
Tested-by: Jenkins
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/utils/lfs.c

index 3a387c4..7c866b3 100644 (file)
@@ -1319,6 +1319,11 @@ static int mirror_create(char *fname, struct mirror_args *mirror_list)
                cur_mirror = cur_mirror->m_next;
        }
 
+       if (layout == NULL) {
+               fprintf(stderr, "error: %s: layout is NULL\n", progname);
+               return -EINVAL;
+       }
+
        rc = llapi_layout_mirror_count_set(layout, mirror_count);
        if (rc) {
                rc = -errno;