[FreeBSD] Plug memory leak in drm_rmdraw() and drm_drawable_free_all()
authorRobert Noland <rnoland@2hip.net>
Fri, 10 Oct 2008 22:23:11 +0000 (18:23 -0400)
committerRobert Noland <rnoland@2hip.net>
Fri, 10 Oct 2008 22:23:11 +0000 (18:23 -0400)
bsd-core/drm_drawable.c

index 018db57..2ae11db 100644 (file)
@@ -71,8 +71,8 @@ int drm_adddraw(struct drm_device *dev, void *data, struct drm_file *file_priv)
        struct drm_draw *draw = data;
        struct bsd_drm_drawable_info *info;
 
-       info = drm_calloc(1, sizeof(struct bsd_drm_drawable_info),
-           DRM_MEM_DRAWABLE);
+       info = malloc(sizeof(struct bsd_drm_drawable_info), DRM_MEM_DRAWABLE,
+           M_NOWAIT | M_ZERO);
        if (info == NULL)
                return ENOMEM;
 
@@ -99,8 +99,8 @@ int drm_rmdraw(struct drm_device *dev, void *data, struct drm_file *file_priv)
                    (struct bsd_drm_drawable_info *)info);
                DRM_SPINUNLOCK(&dev->drw_lock);
                free_unr(dev->drw_unrhdr, draw->handle);
-               drm_free(info, sizeof(struct bsd_drm_drawable_info),
-                   DRM_MEM_DRAWABLE);
+               free(info->rects, DRM_MEM_DRAWABLE);
+               free(info, DRM_MEM_DRAWABLE);
                return 0;
        } else {
                DRM_SPINUNLOCK(&dev->drw_lock);
@@ -123,9 +123,7 @@ int drm_update_draw(struct drm_device *dev, void *data,
        case DRM_DRAWABLE_CLIPRECTS:
                DRM_SPINLOCK(&dev->drw_lock);
                if (update->num != info->num_rects) {
-                       drm_free(info->rects,
-                           sizeof(*info->rects) * info->num_rects,
-                           DRM_MEM_DRAWABLE);
+                       free(info->rects, DRM_MEM_DRAWABLE);
                        info->rects = NULL;
                        info->num_rects = 0;
                }
@@ -134,8 +132,8 @@ int drm_update_draw(struct drm_device *dev, void *data,
                        return 0;
                }
                if (info->rects == NULL) {
-                       info->rects = drm_alloc(sizeof(*info->rects) *
-                           update->num, DRM_MEM_DRAWABLE);
+                       info->rects = malloc(sizeof(*info->rects) *
+                           update->num, DRM_MEM_DRAWABLE, M_NOWAIT);
                        if (info->rects == NULL) {
                                DRM_SPINUNLOCK(&dev->drw_lock);
                                return ENOMEM;
@@ -164,8 +162,8 @@ void drm_drawable_free_all(struct drm_device *dev)
                    (struct bsd_drm_drawable_info *)info);
                DRM_SPINUNLOCK(&dev->drw_lock);
                free_unr(dev->drw_unrhdr, info->handle);
-               drm_free(info, sizeof(struct bsd_drm_drawable_info),
-                   DRM_MEM_DRAWABLE);
+               free(info->info.rects, DRM_MEM_DRAWABLE);
+               free(info, DRM_MEM_DRAWABLE);
                DRM_SPINLOCK(&dev->drw_lock);
        }
        DRM_SPINUNLOCK(&dev->drw_lock);