[PATCH xf86-video-mga 2/4] Remove if(E != NULL) checks around free(E)

Jamey Sharp jamey at minilop.net
Mon Sep 26 21:11:27 PDT 2011


Reviewed-by: Jamey Sharp <jamey at minilop.net>

On Mon, Sep 26, 2011 at 10:41:50PM -0400, Matt Turner wrote:
> Signed-off-by: Matt Turner <mattst88 at gmail.com>
> ---
>  src/mga_dri.c    |   24 ++++++++----------------
>  src/mga_driver.c |   20 ++++++--------------
>  src/mga_merge.c  |    6 ++----
>  src/mga_video.c  |    3 +--
>  4 files changed, 17 insertions(+), 36 deletions(-)
> 
> diff --git a/src/mga_dri.c b/src/mga_dri.c
> index 957c6df..1454994 100644
> --- a/src/mga_dri.c
> +++ b/src/mga_dri.c
> @@ -1011,8 +1011,8 @@ static void MGADRIMoveBuffersXAA(WindowPtr pParent, DDXPointRec ptOldOrg,
>  	    pboxNew2 = (BoxPtr)malloc(sizeof(BoxRec)*nbox);
>  	    pptNew2 = (DDXPointPtr)malloc(sizeof(DDXPointRec)*nbox);
>  	    if (!pboxNew2 || !pptNew2) {
> -	        if (pptNew2) free(pptNew2);
> -	        if (pboxNew2) free(pboxNew2);
> +	        free(pptNew2);
> +	        free(pboxNew2);
>  	        if (pboxNew1) {
>  		    free(pptNew1);
>  		    free(pboxNew1);
> @@ -1494,21 +1494,13 @@ void MGADRICloseScreen( ScreenPtr pScreen )
>     DRICloseScreen( pScreen );
>  
>     if ( pMga->pDRIInfo ) {
> -      if ( pMga->pDRIInfo->devPrivate ) {
> -	 free( pMga->pDRIInfo->devPrivate );
> -	 pMga->pDRIInfo->devPrivate = 0;
> -      }
> +      free(pMga->pDRIInfo->devPrivate);
> +      pMga->pDRIInfo->devPrivate = 0;
>        DRIDestroyInfoRec( pMga->pDRIInfo );
>        pMga->pDRIInfo = 0;
>     }
> -   if ( pMga->DRIServerInfo ) {
> -      free(pMga->DRIServerInfo);
> -      pMga->DRIServerInfo = 0;
> -   }
> -   if ( pMga->pVisualConfigs ) {
> -      free(pMga->pVisualConfigs);
> -   }
> -   if ( pMga->pVisualConfigsPriv ) {
> -      free( pMga->pVisualConfigsPriv );
> -   }
> +   free(pMga->DRIServerInfo);
> +   pMga->DRIServerInfo = 0;
> +   free(pMga->pVisualConfigs);
> +   free(pMga->pVisualConfigsPriv);
>  }
> diff --git a/src/mga_driver.c b/src/mga_driver.c
> index d4ae346..dcbfabf 100644
> --- a/src/mga_driver.c
> +++ b/src/mga_driver.c
> @@ -644,8 +644,6 @@ MGAGetRec(ScrnInfoPtr pScrn)
>  void
>  MGAFreeRec(ScrnInfoPtr pScrn)
>  {
> -    if (pScrn->driverPrivate == NULL)
> -	return;
>      free(pScrn->driverPrivate);
>      pScrn->driverPrivate = NULL;
>  }
> @@ -2430,8 +2428,7 @@ MGAPreInit(ScrnInfoPtr pScrn, int flags)
>  			      pMga->FbMapSize,
>  			      LOOKUP_BEST_REFRESH);
>        
> -	if (linePitches)
> -	   free(linePitches);
> +	free(linePitches);
>      }
>  
>      if (i < 1 && pMga->FBDev) {
> @@ -3750,16 +3747,11 @@ MGACloseScreen(int scrnIndex, ScreenPtr pScreen)
>  #endif
>      if (pMga->CursorInfoRec)
>      	xf86DestroyCursorInfoRec(pMga->CursorInfoRec);
> -    if (pMga->ShadowPtr)
> -	free(pMga->ShadowPtr);
> -    if (pMga->DGAModes)
> -	free(pMga->DGAModes);
> -    if (pMga->adaptor)
> -	free(pMga->adaptor);
> -    if (pMga->portPrivate)
> -	free(pMga->portPrivate);
> -    if (pMga->ScratchBuffer)
> -	free(pMga->ScratchBuffer);
> +    free(pMga->ShadowPtr);
> +    free(pMga->DGAModes);
> +    free(pMga->adaptor);
> +    free(pMga->portPrivate);
> +    free(pMga->ScratchBuffer);
>  
>      pScrn->vtSema = FALSE;
>  
> diff --git a/src/mga_merge.c b/src/mga_merge.c
> index e3216c9..c1cf2eb 100644
> --- a/src/mga_merge.c
> +++ b/src/mga_merge.c
> @@ -528,8 +528,7 @@ MGAPreInitMergedFB(ScrnInfoPtr pScrn1, int flags)
>  			      pMga->FbMapSize,
>  			      LOOKUP_BEST_REFRESH);
>          
> -	if (linePitches)
> -	   free(linePitches);
> +	free(linePitches);
>      }
>  
>  
> @@ -892,8 +891,7 @@ MGACloseScreenMerged(int scrnIndex, ScreenPtr pScreen) {
>          pScrn1->currentMode = pScrn1->modes;
>          do {
>              DisplayModePtr p = pScrn1->currentMode->next; 
> -            if(pScrn1->currentMode->Private) 
> -                free(pScrn1->currentMode->Private);
> +            free(pScrn1->currentMode->Private);
>              free(pScrn1->currentMode);
>              pScrn1->currentMode = p;
>          }while( pScrn1->currentMode != pScrn1->modes);
> diff --git a/src/mga_video.c b/src/mga_video.c
> index 52b2e16..b7f58c5 100644
> --- a/src/mga_video.c
> +++ b/src/mga_video.c
> @@ -145,8 +145,7 @@ void MGAInitVideo(ScreenPtr pScreen)
>      if(num_adaptors)
>          xf86XVScreenInit(pScreen, adaptors, num_adaptors);
>  
> -    if(newAdaptors)
> -	free(newAdaptors);
> +    free(newAdaptors);
>  }
>  
>  /* client libraries expect an encoding */
> -- 
> 1.7.3.4
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110926/054861fa/attachment.pgp>


More information about the xorg-devel mailing list