diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-11-13 11:58:26 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-11-16 12:25:53 -0800 |
| commit | 3251627c943f1cfc5bfad7c38d4e8d85cf44c6d8 (patch) | |
| tree | c64fb446d9c778499a494bb06711affbae88f446 /drivers/staging/intel_sst/intel_sst_stream_encoded.c | |
| parent | 832855354b5863d141ecfc563e78063978c6f29f (diff) | |
| download | op-kernel-dev-3251627c943f1cfc5bfad7c38d4e8d85cf44c6d8.zip op-kernel-dev-3251627c943f1cfc5bfad7c38d4e8d85cf44c6d8.tar.gz | |
Staging: intel_sst: fix memory leak
The original code set "str_info->decode_ibuf" to NULL so the kfree() is
no-op.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Harsha Priya <priya.harsha@intel.com>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/intel_sst/intel_sst_stream_encoded.c')
| -rw-r--r-- | drivers/staging/intel_sst/intel_sst_stream_encoded.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/intel_sst/intel_sst_stream_encoded.c b/drivers/staging/intel_sst/intel_sst_stream_encoded.c index fbae39f..5c45560 100644 --- a/drivers/staging/intel_sst/intel_sst_stream_encoded.c +++ b/drivers/staging/intel_sst/intel_sst_stream_encoded.c @@ -1269,7 +1269,7 @@ finish: dbufs->output_bytes_produced = total_output; str_info->status = str_info->prev; str_info->prev = STREAM_DECODE; - str_info->decode_ibuf = NULL; kfree(str_info->decode_ibuf); + str_info->decode_ibuf = NULL; return retval; } |
