# HG changeset patch # User reimar # Date 1225457957 0 # Node ID 28b6709b52a289467eee7283561c6bb361142498 # Parent d35bcab9833bf99f94580bf0215a1d3d0eb26f0a Avoid a potential memleak in parse_obj_params in case of a missing m_ob_params_t part. Fixes bug #1318. diff -r d35bcab9833b -r 28b6709b52a2 m_option.c --- a/m_option.c Fri Oct 31 12:56:25 2008 +0000 +++ b/m_option.c Fri Oct 31 12:59:17 2008 +0000 @@ -1476,13 +1476,14 @@ int r; m_obj_params_t* p = opt->priv; const m_struct_t* desc; - char* cpy = strdup(param); + char* cpy; // We need the object desc if(!p) return M_OPT_INVALID; desc = p->desc; + cpy = strdup(param); r = get_obj_params(name,desc->name,cpy,desc,p->separator,dst ? &opts : NULL); free(cpy); if(r < 0)