Remove NULL pointer dereference

Bug found by the "andand" semantic patch:

// The right argument of || or && is dereferencing something known to be NULL
//
// Confidence: High
// Copyright: (C) Gilles Muller, Julia Lawall, EMN, INRIA, DIKU.  GPLv2.
// URL: http://coccinelle.lip6.fr/rules/andand.html
// Options:

@ expression@
expression E;
identifier fld;
@@

- !E &&
+ !E ||
  <+...E->fld...+>

@ expression@
expression E;
identifier fld;
@@

- E ||
+ E &&
  <+...E->fld...+>




SVN revision: 50118
This commit is contained in:
Lucas De Marchi 2010-07-08 03:21:33 +00:00
parent e9d4afc9d7
commit 9df58ae530
1 changed files with 2 additions and 2 deletions

View File

@ -807,7 +807,7 @@ eng_image_data_preload_request(void *data __UNUSED__, void *image, const void *t
{
Evas_Quartz_Image *im = (Evas_Quartz_Image *)image;
if (!im && !im->im) return ;
if (!im || !im->im) return ;
evas_cache_image_preload_data(&im->im->cache_entry, target);
}
@ -816,7 +816,7 @@ eng_image_data_preload_cancel(void *data __UNUSED__, void *image, const void *ta
{
Evas_Quartz_Image *im = (Evas_Quartz_Image *)image;
if (!im && !im->im) return ;
if (!im || !im->im) return ;
evas_cache_image_preload_cancel(&im->im->cache_entry, target);
}