Messages in this thread | | | Date | Tue, 22 Jan 2013 23:59:23 -0600 | Subject | Re: [PATCH] fs/cifs/cifs_dfs_ref.c: fix potential memory leakage | From | Steve French <> |
| |
merged into cifs-2.6.git
On Tue, Jan 22, 2013 at 7:36 PM, Jeff Layton <jlayton@redhat.com> wrote: > On Tue, 22 Jan 2013 19:20:58 -0500 > Cong Ding <dinggnu@gmail.com> wrote: > >> When it goes to error through line 144, the memory allocated to *devname is >> not freed, and the caller doesn't free it either in line 250. So we free the >> memroy of *devname in function cifs_compose_mount_options() when it goes to >> error. >> >> Signed-off-by: Cong Ding <dinggnu@gmail.com> >> --- >> fs/cifs/cifs_dfs_ref.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/fs/cifs/cifs_dfs_ref.c b/fs/cifs/cifs_dfs_ref.c >> index ce5cbd7..210fce2 100644 >> --- a/fs/cifs/cifs_dfs_ref.c >> +++ b/fs/cifs/cifs_dfs_ref.c >> @@ -226,6 +226,8 @@ compose_mount_options_out: >> compose_mount_options_err: >> kfree(mountdata); >> mountdata = ERR_PTR(rc); >> + kfree(*devname); >> + *devname = NULL; >> goto compose_mount_options_out; >> } >> > > Nice catch. Looks correct to me. Should this also go to stable? > > Reviewed-by: Jeff Layton <jlayton@redhat.com>
-- Thanks,
Steve
| |