merged into cifs-2.6.git for-next On Wed, Aug 20, 2014 at 5:39 AM, Namjae Jeon <[email protected]> wrote: > Unlikely but possible. When password is supplied multiple times, we have > to free the previous allocation. > > Signed-off-by: Namjae Jeon <[email protected]> > Signed-off-by: Ashish Sangwan <[email protected]> > --- > fs/cifs/connect.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c > index b4b6d10..91512fd 100644 > --- a/fs/cifs/connect.c > +++ b/fs/cifs/connect.c > @@ -1583,6 +1583,7 @@ cifs_parse_mount_options(const char *mountdata, const > char *devname, > * Check if this is a case where the password > * starts with a delimiter > */ > + kfree(vol->password); > tmp_end = strchr(data, '='); > tmp_end++; > if (!(tmp_end < end && tmp_end[1] == delim)) { > @@ -1624,6 +1625,7 @@ cifs_parse_mount_options(const char *mountdata, const > char *devname, > options = end; > } > > + kfree(vol->password); > /* Now build new password string */ > temp_len = strlen(value); > vol->password = kzalloc(temp_len+1, GFP_KERNEL); > -- > 1.7.7 >
-- Thanks, Steve -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
