diff options
| author | Brian Paul <brianp@vmware.com> | 2016-04-05 09:56:49 -0600 |
|---|---|---|
| committer | Brian Paul <brianp@vmware.com> | 2016-04-07 11:42:43 -0600 |
| commit | 9f443af449515240169e69b98bdb3746f38e7f35 (patch) | |
| tree | 567451e27c63154d471ec2bf657033b81544b345 /src/gallium/drivers/svga | |
| parent | 60cf2fa477c1a91c1f8daea14d14edca80f1e183 (diff) | |
| download | external_mesa3d-9f443af449515240169e69b98bdb3746f38e7f35.zip external_mesa3d-9f443af449515240169e69b98bdb3746f38e7f35.tar.gz external_mesa3d-9f443af449515240169e69b98bdb3746f38e7f35.tar.bz2 | |
svga: add some trivial null pointer checks
These small mallocs will probably never fail, but static analysis tools
may complain about the missing checks.
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
Diffstat (limited to 'src/gallium/drivers/svga')
| -rw-r--r-- | src/gallium/drivers/svga/svga_pipe_blend.c | 3 | ||||
| -rw-r--r-- | src/gallium/drivers/svga/svga_pipe_depthstencil.c | 3 | ||||
| -rw-r--r-- | src/gallium/drivers/svga/svga_pipe_rasterizer.c | 3 |
3 files changed, 9 insertions, 0 deletions
diff --git a/src/gallium/drivers/svga/svga_pipe_blend.c b/src/gallium/drivers/svga/svga_pipe_blend.c index 0af80cd..0ba9313 100644 --- a/src/gallium/drivers/svga/svga_pipe_blend.c +++ b/src/gallium/drivers/svga/svga_pipe_blend.c @@ -142,6 +142,9 @@ svga_create_blend_state(struct pipe_context *pipe, struct svga_blend_state *blend = CALLOC_STRUCT( svga_blend_state ); unsigned i; + if (!blend) + return NULL; + /* Fill in the per-rendertarget blend state. We currently only * support independent blend enable and colormask per render target. */ diff --git a/src/gallium/drivers/svga/svga_pipe_depthstencil.c b/src/gallium/drivers/svga/svga_pipe_depthstencil.c index d84ed1d..83fcdc3 100644 --- a/src/gallium/drivers/svga/svga_pipe_depthstencil.c +++ b/src/gallium/drivers/svga/svga_pipe_depthstencil.c @@ -134,6 +134,9 @@ svga_create_depth_stencil_state(struct pipe_context *pipe, struct svga_context *svga = svga_context(pipe); struct svga_depth_stencil_state *ds = CALLOC_STRUCT( svga_depth_stencil_state ); + if (!ds) + return NULL; + /* Don't try to figure out CW/CCW correspondence with * stencil[0]/[1] at this point. Presumably this can change as * back/front face are modified. diff --git a/src/gallium/drivers/svga/svga_pipe_rasterizer.c b/src/gallium/drivers/svga/svga_pipe_rasterizer.c index 8e0db53..d397c95 100644 --- a/src/gallium/drivers/svga/svga_pipe_rasterizer.c +++ b/src/gallium/drivers/svga/svga_pipe_rasterizer.c @@ -161,6 +161,9 @@ svga_create_rasterizer_state(struct pipe_context *pipe, struct svga_rasterizer_state *rast = CALLOC_STRUCT( svga_rasterizer_state ); struct svga_screen *screen = svga_screen(pipe->screen); + if (!rast) + return NULL; + /* need this for draw module. */ rast->templ = *templ; |