Skip to content

Commit

Permalink
fbdev: broadsheetfb: fix memory leak
Browse files Browse the repository at this point in the history
We have requested the firmware and it was loaded but we missed releasing
it both on success and error.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
  • Loading branch information
Sudip Mukherjee authored and Tomi Valkeinen committed Sep 30, 2015
1 parent 4bdb462 commit 4d370b7
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions drivers/video/fbdev/broadsheetfb.c
Original file line number Diff line number Diff line change
Expand Up @@ -752,7 +752,7 @@ static ssize_t broadsheet_loadstore_waveform(struct device *dev,
if ((fw_entry->size < 8*1024) || (fw_entry->size > 64*1024)) {
dev_err(dev, "Invalid waveform\n");
err = -EINVAL;
goto err_failed;
goto err_fw;
}

mutex_lock(&(par->io_lock));
Expand All @@ -762,13 +762,15 @@ static ssize_t broadsheet_loadstore_waveform(struct device *dev,
mutex_unlock(&(par->io_lock));
if (err < 0) {
dev_err(dev, "Failed to store broadsheet waveform\n");
goto err_failed;
goto err_fw;
}

dev_info(dev, "Stored broadsheet waveform, size %zd\n", fw_entry->size);

return len;
err = len;

err_fw:
release_firmware(fw_entry);
err_failed:
return err;
}
Expand Down

0 comments on commit 4d370b7

Please sign in to comment.