diff options
| author | Rob Clark <robclark@freedesktop.org> | 2014-05-04 10:40:46 -0400 |
|---|---|---|
| committer | Rob Clark <robclark@freedesktop.org> | 2014-05-04 11:08:10 -0400 |
| commit | b7e7ae9f605283076dba6d9e826fbbd1f1db1c4b (patch) | |
| tree | d15bf50b7ca2c7b18597d146b7d3cbd8898b6e02 /src/gallium/state_trackers/xa | |
| parent | 829cb0423d18331b9ba0b6098e2b312ff2ce6053 (diff) | |
| download | external_mesa3d-b7e7ae9f605283076dba6d9e826fbbd1f1db1c4b.zip external_mesa3d-b7e7ae9f605283076dba6d9e826fbbd1f1db1c4b.tar.gz external_mesa3d-b7e7ae9f605283076dba6d9e826fbbd1f1db1c4b.tar.bz2 | |
xa: fix segfault
Fixes:
Program received signal SIGSEGV, Segmentation fault.
bind_samplers (comp=0x21b054, comp=0x21b054, ctx=0x211430)
at ../../../../../src/gallium/state_trackers/xa/xa_composite.c:445
445 mask_pic->srf->tex->format);
(gdb) bt
#0 bind_samplers (comp=0x21b054, comp=0x21b054, ctx=0x211430)
at ../../../../../src/gallium/state_trackers/xa/xa_composite.c:445
#1 xa_composite_prepare (ctx=0x211430, comp=comp@entry=0x21b054)
at ../../../../../src/gallium/state_trackers/xa/xa_composite.c:488
#2 0xb6f454b4 in XAPrepareComposite (op=<optimized out>, pSrcPicture=<optimized out>,
pMaskPicture=<optimized out>, pDstPicture=<optimized out>, pSrc=0x5b3ad8, pMask=0x0,
pDst=0x5923b8) at msm-exa-xa.c:533
We can't yet handle solid fill mask, so explicitly reject that, rather
than segfaulting. Otherwise DDX would need to check XA version to see
if solid fill mask were supported.
Signed-off-by: Rob Clark <robclark@freedesktop.org>
Diffstat (limited to 'src/gallium/state_trackers/xa')
| -rw-r--r-- | src/gallium/state_trackers/xa/xa_composite.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/gallium/state_trackers/xa/xa_composite.c b/src/gallium/state_trackers/xa/xa_composite.c index 7ae35a1..c283a0d 100644 --- a/src/gallium/state_trackers/xa/xa_composite.c +++ b/src/gallium/state_trackers/xa/xa_composite.c @@ -472,6 +472,9 @@ xa_composite_prepare(struct xa_context *ctx, struct xa_surface *dst_srf = comp->dst->srf; int ret; + if (comp->mask && !comp->mask->srf) + return -XA_ERR_INVAL; + ret = xa_ctx_srf_create(ctx, dst_srf); if (ret != XA_ERR_NONE) return ret; |