fix linux memory leak #3 (#4)

bithavoc · Dec 7, 2017 · 32b9812 · 32b9812
1 parent 46e2ee6
commit 32b9812
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -22,6 +22,12 @@ var desktopIdle = require('desktop-idle');
 console.log(desktopIdle.getIdleTime());
 ```
 
+### Linux Requirements
+
+X server development package and pkg-config are required:
+
+`apt install libxss-dev pkg-config`
+
 ### Test
 
 ```

diff --git a/src/linux/idle.cc b/src/linux/idle.cc
@@ -8,5 +8,8 @@ double desktop_idle_get_time() {
   }
   XScreenSaverInfo *info = XScreenSaverAllocInfo();
   XScreenSaverQueryInfo(dpy, DefaultRootWindow(dpy), info);
-  return info->idle / 1000; // to seconds
+  unsigned long idle = info->idle;
+  XFree(info);
+  XCloseDisplay(dpy);
+  return idle / 1000; // to seconds
 }