ecore_drm: pass NULL to mmap() instead 0 for address
authorStefan Schmidt <stefan@osg.samsung.com>
Tue, 10 Nov 2015 15:31:36 +0000 (16:31 +0100)
committerStefan Schmidt <stefan@osg.samsung.com>
Tue, 10 Nov 2015 15:34:13 +0000 (16:34 +0100)
The second find of these. The other one was in evas generic loaders.
mmap() expects a void * as address here. If we want to let the kernel
choose the address we should really pass in NULL instead of the
integer 0.

Thanks goes to the sparse semantic parser for pointing this out.

src/lib/ecore_drm/ecore_drm_fb.c

index 385d3ea..00c062a 100644 (file)
@@ -99,7 +99,7 @@ ecore_drm_fb_create(Ecore_Drm_Device *dev, int width, int height)
      }
 
    fb->mmap =
-     mmap(0, fb->size, PROT_WRITE, MAP_SHARED, dev->drm.fd, marg.offset);
+     mmap(NULL, fb->size, PROT_WRITE, MAP_SHARED, dev->drm.fd, marg.offset);
    if (fb->mmap == MAP_FAILED)
      {
         ERR("Could not mmap framebuffer space: %m");