summaryrefslogtreecommitdiffstats
path: root/sys/dev/drm/radeon_state.c
diff options
context:
space:
mode:
authorsam <sam@FreeBSD.org>2005-02-26 18:52:54 +0000
committersam <sam@FreeBSD.org>2005-02-26 18:52:54 +0000
commit23f2d2ade46da4c64513b325284f57076d1a23f2 (patch)
treed3b122f462420937d4cb6ea316c9e718f7408f57 /sys/dev/drm/radeon_state.c
parent93e61553f4dc84ce7e0c53eb383665227305d217 (diff)
downloadFreeBSD-src-23f2d2ade46da4c64513b325284f57076d1a23f2.zip
FreeBSD-src-23f2d2ade46da4c64513b325284f57076d1a23f2.tar.gz
move ptr deref's to after null checks
Noticed by: Coverity Prevent analysis tool
Diffstat (limited to 'sys/dev/drm/radeon_state.c')
-rw-r--r--sys/dev/drm/radeon_state.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/sys/dev/drm/radeon_state.c b/sys/dev/drm/radeon_state.c
index e6caf2f..70e1b07 100644
--- a/sys/dev/drm/radeon_state.c
+++ b/sys/dev/drm/radeon_state.c
@@ -1658,7 +1658,7 @@ int radeon_cp_vertex( DRM_IOCTL_ARGS )
DRM_DEVICE;
drm_radeon_private_t *dev_priv = dev->dev_private;
drm_file_t *filp_priv;
- drm_radeon_sarea_t *sarea_priv = dev_priv->sarea_priv;
+ drm_radeon_sarea_t *sarea_priv;
drm_device_dma_t *dma = dev->dma;
drm_buf_t *buf;
drm_radeon_vertex_t vertex;
@@ -1670,6 +1670,7 @@ int radeon_cp_vertex( DRM_IOCTL_ARGS )
DRM_ERROR( "%s called with no initialization\n", __FUNCTION__ );
return DRM_ERR(EINVAL);
}
+ sarea_priv = dev_priv->sarea_priv;
DRM_GET_PRIV_WITH_RETURN( filp_priv, filp );
@@ -1748,7 +1749,7 @@ int radeon_cp_indices( DRM_IOCTL_ARGS )
DRM_DEVICE;
drm_radeon_private_t *dev_priv = dev->dev_private;
drm_file_t *filp_priv;
- drm_radeon_sarea_t *sarea_priv = dev_priv->sarea_priv;
+ drm_radeon_sarea_t *sarea_priv;
drm_device_dma_t *dma = dev->dma;
drm_buf_t *buf;
drm_radeon_indices_t elts;
@@ -1761,6 +1762,7 @@ int radeon_cp_indices( DRM_IOCTL_ARGS )
DRM_ERROR( "%s called with no initialization\n", __FUNCTION__ );
return DRM_ERR(EINVAL);
}
+ sarea_priv = dev_priv->sarea_priv;
DRM_GET_PRIV_WITH_RETURN( filp_priv, filp );
@@ -1979,7 +1981,7 @@ int radeon_cp_vertex2( DRM_IOCTL_ARGS )
DRM_DEVICE;
drm_radeon_private_t *dev_priv = dev->dev_private;
drm_file_t *filp_priv;
- drm_radeon_sarea_t *sarea_priv = dev_priv->sarea_priv;
+ drm_radeon_sarea_t *sarea_priv;
drm_device_dma_t *dma = dev->dma;
drm_buf_t *buf;
drm_radeon_vertex2_t vertex;
@@ -1992,6 +1994,7 @@ int radeon_cp_vertex2( DRM_IOCTL_ARGS )
DRM_ERROR( "%s called with no initialization\n", __FUNCTION__ );
return DRM_ERR(EINVAL);
}
+ sarea_priv = dev_priv->sarea_priv;
DRM_GET_PRIV_WITH_RETURN( filp_priv, filp );
OpenPOWER on IntegriCloud