diff options
| author | Jubeom Kim <jubeomk@nvidia.com> | 2011-06-22 22:56:21 +0900 |
|---|---|---|
| committer | Manish Tuteja <mtuteja@nvidia.com> | 2011-07-15 12:05:52 -0700 |
| commit | 30e5d005bf7ce85f4df7af17d690a0353b65a2ec (patch) | |
| tree | 861b280d9f849aaeab301a06675471acc53d72db /drivers | |
| parent | c560574fcb4d8c58a4554021793e09358658fd4f (diff) | |
tegra_mediaserver: fix wrong kzalloc param, uninitialized values
- Changed the kzalloc param from node to block.
- 'CHECK_CONDITION((physical_address < 0), ...' is not valid,
because the physical_address value is unsigned value.
- Fixed uninitialized values
Bug 825511
Change-Id: Ib579c591c39396533eea8ecc42e92373ed49bd18
Reviewed-on: http://git-master/r/37862
(cherry picked from commit 06d905e9ed14d28224d5f59acf75b4de503f8796)
Reviewed-on: http://git-master/r/39782
Tested-by: Shridhar Rasal <srasal@nvidia.com>
Reviewed-by: Jubeom Kim <jubeomk@nvidia.com>
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/media/video/tegra/mediaserver/tegra_mediaserver.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/media/video/tegra/mediaserver/tegra_mediaserver.c b/drivers/media/video/tegra/mediaserver/tegra_mediaserver.c index e25e1926d99e..a26b9e990ca1 100644 --- a/drivers/media/video/tegra/mediaserver/tegra_mediaserver.c +++ b/drivers/media/video/tegra/mediaserver/tegra_mediaserver.c @@ -244,7 +244,7 @@ static int mediasrv_alloc(struct tegra_mediasrv_node *node, { struct tegra_mediasrv_block *block; - block = kzalloc(sizeof(struct tegra_mediasrv_node), + block = kzalloc(sizeof(struct tegra_mediasrv_block), GFP_KERNEL); CHECK_NULL(block, block_alloc_fail); @@ -269,7 +269,8 @@ block_done: if (!mediasrv->nr_iram_shared) { size_t align, size; struct nvmap_handle_ref *r = NULL; - unsigned long id, physical_address; + unsigned long id; + int physical_address; size = PAGE_ALIGN(in->in.u.iram.size); r = nvmap_create_handle(mediasrv->nvmap, size); @@ -339,7 +340,7 @@ static void mediasrv_free(struct tegra_mediasrv_node *node, switch (in->in.tegra_mediaserver_resource_type) { case TEGRA_MEDIASERVER_RESOURCE_BLOCK: { - struct tegra_mediasrv_block *block; + struct tegra_mediasrv_block *block = NULL; struct tegra_mediasrv_block *temp; struct list_head *entry; @@ -392,7 +393,7 @@ static int mediasrv_update_block_info( union tegra_mediaserver_update_block_info *in ) { - struct tegra_mediasrv_block *entry; + struct tegra_mediasrv_block *entry = NULL; struct tegra_mediasrv_block *block; int e; |