X-Git-Url: https://git.danieliu.xyz/?a=blobdiff_plain;f=slock.c;h=089b9505071f06df18add22225aa496a89751d8d;hb=663fdb326a573ed8e4a5e47a6d8459b2f479b75f;hp=6541b7046cc5c7a50a5c6bda7ef7581e616eec38;hpb=27d8d248275ca3eb63882202dcc21ecf2f81e72a;p=slock.git

diff --git a/slock.c b/slock.c
index 6541b70..089b950 100644
--- a/slock.c
+++ b/slock.c
@@ -1,7 +1,10 @@
-/* (C)opyright MMIV-MMV Anselm R. Garbe <garbeam at gmail dot com>
+/* (C)opyright MMVI-MMVII Anselm R. Garbe <garbeam at gmail dot com>
  * See LICENSE file for license details.
  */
 #define _XOPEN_SOURCE 500
+#if HAVE_SHADOW_H
+#include <shadow.h>
+#endif
 
 #include <ctype.h>
 #include <pwd.h>
@@ -9,9 +12,6 @@
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>
-#if HAVE_SHADOW_H
-#include <shadow.h>
-#endif
 #include <sys/types.h>
 #include <X11/keysym.h>
 #include <X11/Xlib.h>
@@ -75,8 +75,6 @@ main(int argc, char **argv) {
 	screen = DefaultScreen(dpy);
 
 	/* init */
-	len = 0;
-
 	wa.override_redirect = 1;
 	wa.background_pixel = BlackPixel(dpy, screen);
 	w = XCreateWindow(dpy, RootWindow(dpy, screen), 0, 0,
@@ -94,8 +92,9 @@ main(int argc, char **argv) {
 	len = 10;
 	for(len = 10; len && (XGrabKeyboard(dpy, RootWindow(dpy, screen), True, GrabModeAsync,
 		GrabModeAsync, CurrentTime) != GrabSuccess); len--)
-		usleep(1000);
+		usleep(10000);
 	running = running && (len > 0);
+	len = 0;
 	XMapRaised(dpy, w);
 	XSync(dpy, False);
 
@@ -103,7 +102,7 @@ main(int argc, char **argv) {
 	while(running && !XNextEvent(dpy, &ev))
 		if(ev.type == KeyPress) {
 			buf[0] = 0;
-			num = XLookupString(&ev.xkey, buf, sizeof(buf), &ksym, 0);
+			num = XLookupString(&ev.xkey, buf, sizeof buf, &ksym, 0);
 			if(IsFunctionKey(ksym) || IsKeypadKey(ksym)
 					|| IsMiscFunctionKey(ksym) || IsPFKey(ksym)
 					|| IsPrivateKeypadKey(ksym))
@@ -120,11 +119,11 @@ main(int argc, char **argv) {
 				break;
 			case XK_BackSpace:
 				if(len)
-				  --len;
+					--len;
 				break;
 			default:
-				if(num && !iscntrl((int) buf[0])) {
-					memcpy(passwd + len,buf,num);
+				if(num && !iscntrl((int) buf[0]) && (len + num < sizeof passwd)) { 
+					memcpy(passwd + len, buf, num);
 					len += num;
 				}
 				break;