diff options
| author | Tuomas Tynkkynen <ttynkkynen@nvidia.com> | 2012-08-20 17:26:16 +0300 |
|---|---|---|
| committer | Simone Willett <swillett@nvidia.com> | 2012-08-22 12:04:09 -0700 |
| commit | adf925fb2e1bf0c25e4427884260bf69780ff0bd (patch) | |
| tree | 404bd7ea1b797efa51c703c44f4976fad09430db /kernel/itimer.c | |
| parent | 6c679d9b3285f60df9acc335b19c1478f8ee92e3 (diff) | |
video: tegra: host: Fix freeing a ERR_PTR value
nvhost_3dctx_alloc_common correctly checks the return value of an
allocator function, and attempts to perform cleanup. However, this
particular allocation API returns ERR_PTR values in case of failure,
which then gets passed to the deallocation function, causing a crash.
Also, the return value of that allocator API is checked with IS_ERR
in some places, and with IS_ERR_OR_NULL in others. For consistency
and avoiding surprises if the allocation implementation changes,
use IS_ERR_OR_NULL in all places.
Bug 1035878
Change-Id: I65ce97e11f9712b2e9630b38183f6e70bc14b254
Signed-off-by: Tuomas Tynkkynen <ttynkkynen@nvidia.com>
Reviewed-on: http://git-master/r/124657
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
Diffstat (limited to 'kernel/itimer.c')
0 files changed, 0 insertions, 0 deletions