[PATCH] - Fix Unchecked memory usage.

Edward O'Callaghan eocallaghan at auroraux.org
Wed Jul 8 09:28:21 PDT 2009


Thanks Jeremy, your quite right.

I normally just set pointers as NULL without thinking too much about
it, figured its a good habit to have. :)

Although it does not matter here, but you can remove the line if you
really want.

Have a good afternoon,
Cheers,
Edward.

2009/7/8 Jeremy Huddleston <jeremyhu at freedesktop.org>:
> Maarten Maathuis wrote:
>> Imagine what happens if everyone sent links. A patch is forgotten or
>> maybe it's part a larger discussion. Suddenly the link dies, and part
>> of the history is gone. It's really annoying to check mailinglist
>> archives only to find dead links.
>>
>> Maarten.
>>
>
> And it makes quoting in a reply easier...
>
> You don't need to initialize bufmgr_gem to NULL here since the first
> thing you do is call calloc() for it... of course any decent optimized
> compiler will simply ignore that '= NULL', but I figured I'd point it out.
>
> @@ -1389,13 +1395,16 @@ drm_intel_gem_bo_disable_reuse(drm_intel_bo *bo)
>  drm_intel_bufmgr *
>  drm_intel_bufmgr_gem_init(int fd, int batch_size)
>  {
> -    drm_intel_bufmgr_gem *bufmgr_gem;
> +    drm_intel_bufmgr_gem *bufmgr_gem = NULL;
>     struct drm_i915_gem_get_aperture aperture;
>     drm_i915_getparam_t gp;
>     int ret, i;
>     unsigned long size;
>
>     bufmgr_gem = calloc(1, sizeof(*bufmgr_gem));
> +    if (bufmgr_gem == NULL) {
> +        return NULL;
> +    }
>     bufmgr_gem->fd = fd;
>
>     if (pthread_mutex_init(&bufmgr_gem->lock, NULL) != 0) {
>
>
>



-- 
-- 
Edward O'Callaghan
http://www.auroraux.org/
eocallaghan at auroraux dot org


More information about the xorg-devel mailing list