diff options
| author | Sameer Wadgaonkar <[email protected]> | 2017-08-30 13:36:09 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2017-08-31 18:17:38 +0200 |
| commit | 90544cb10caafa6c12315215486a62a8d487f6be (patch) | |
| tree | b63c17ba48fb1d0ceb092169ff79ec30ce77094b | |
| parent | 7126395e607ca122aed1f2bcced3075756bb1a54 (diff) | |
staging: unisys: visorbus: visorchipset.c: Fix bug in parser_init_byte_stream.
This patch fixes a bug in the function parser_init_byte_stream()
by removing the call to parser_done from goto err_finish_ctx.
The function parser_done() decrements
chipset_dev->controlvm_payload_bytes_buffered which is not
incremented before this gets called.
Signed-off-by: Sameer Wadgaonkar <[email protected]>
Reported-by: Dan Carpenter <[email protected]>
Signed-off-by: David Kershner <[email protected]>
Reviewed-by: Tim Sell <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/unisys/visorbus/visorchipset.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/unisys/visorbus/visorchipset.c b/drivers/staging/unisys/visorbus/visorchipset.c index 25a30a4b46e4..e296df78c835 100644 --- a/drivers/staging/unisys/visorbus/visorchipset.c +++ b/drivers/staging/unisys/visorbus/visorchipset.c @@ -1505,7 +1505,7 @@ static struct parser_context *parser_init_byte_stream(u64 addr, u32 bytes, return ctx; err_finish_ctx: - parser_done(ctx); + kfree(ctx); return NULL; } |