Whamcloud - gitweb
LU-6828 lod: fix memory leak in lod_connect_to_osd 35/15635/3
authorYang Sheng <yang.sheng@intel.com>
Wed, 22 Jul 2015 01:38:04 +0000 (18:38 -0700)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 29 Jul 2015 15:39:06 +0000 (15:39 +0000)
We would replace '$fsname-mdtlov' to '$fsname-MDT0000-osd'
when upgrade from 1.x. But buffer alloced as former length.

Signed-off-by: Yang Sheng <yang.sheng@intel.com>
Change-Id: I8c92e3cb4472527fa5ad5da7f0e4cb498eaccb6c
Reviewed-on: http://review.whamcloud.com/15635
Tested-by: Jenkins
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/lod/lod_dev.c

index ff75841..dc30788 100644 (file)
@@ -1322,7 +1322,7 @@ static int lod_connect_to_osd(const struct lu_env *env, struct lod_device *lod,
         * we use "-MDT" to differentiate 2.x from 1.8 */
 
        if ((p = lustre_cfg_string(cfg, 0)) && strstr(p, "-mdtlov")) {
-               len = strlen(p) + 1;
+               len = strlen(p) + 6;
                OBD_ALLOC(nextdev, len);
                if (nextdev == NULL)
                        GOTO(out, rc = -ENOMEM);