diff options
| author | Jakob Bornecrantz <jakob@vmware.com> | 2013-10-30 05:46:56 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-04 13:57:05 -0500 |
| commit | f692d2d8bebddeb8c8faedfc41e4732f3dc23e5b (patch) | |
| tree | b8d656e6a7060b0483ffca2c4b04f0ad8eb1065b /drivers/gpu | |
| parent | 4f94b6e4b43ca3a1027803bd3cab21e72bbb991e (diff) | |
drm/ttm: Handle in-memory region copies
commit 9a0599ddeae012a771bba5e23393fc52d8a59d89 upstream.
Fix the case where the ttm pointer may be NULL causing
a NULL pointer dereference.
Signed-off-by: Jakob Bornecrantz <jakob@vmware.com>
Signed-off-by: Thomas Hellström <thellstrom@vmware.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/ttm/ttm_bo_util.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c index af894584dd90..068341ad1f5f 100644 --- a/drivers/gpu/drm/ttm/ttm_bo_util.c +++ b/drivers/gpu/drm/ttm/ttm_bo_util.c | |||
| @@ -347,7 +347,9 @@ int ttm_bo_move_memcpy(struct ttm_buffer_object *bo, | |||
| 347 | if (old_iomap == NULL && ttm == NULL) | 347 | if (old_iomap == NULL && ttm == NULL) |
| 348 | goto out2; | 348 | goto out2; |
| 349 | 349 | ||
| 350 | if (ttm->state == tt_unpopulated) { | 350 | /* TTM might be null for moves within the same region. |
| 351 | */ | ||
| 352 | if (ttm && ttm->state == tt_unpopulated) { | ||
| 351 | ret = ttm->bdev->driver->ttm_tt_populate(ttm); | 353 | ret = ttm->bdev->driver->ttm_tt_populate(ttm); |
| 352 | if (ret) { | 354 | if (ret) { |
| 353 | /* if we fail here don't nuke the mm node | 355 | /* if we fail here don't nuke the mm node |
