[BusyBox-cvs] busybox/console-tools loadkmap.c,1.26,1.27

Glenn McGrath bug1 at busybox.net
Fri Nov 21 09:27:07 UTC 2003


Update of /var/cvs/busybox/console-tools
In directory winder:/tmp/cvs-serv1496/console-tools

Modified Files:
	loadkmap.c 
Log Message:
Make use of libbb functions bb_xopen, bb_full_read, used #define's 
instead of static consts, avoid xmalloc.


Index: loadkmap.c
===================================================================
RCS file: /var/cvs/busybox/console-tools/loadkmap.c,v
retrieving revision 1.26
retrieving revision 1.27
diff -u -d -r1.26 -r1.27
--- loadkmap.c	19 Mar 2003 09:11:29 -0000	1.26
+++ loadkmap.c	21 Nov 2003 09:27:02 -0000	1.27
@@ -37,43 +37,35 @@
 	unsigned char kb_index;
 	unsigned short kb_value;
 };
-static const int KDSKBENT = 0x4B47;  /* sets one entry in translation table */
+/* sets one entry in translation table */
+#define KDSKBENT        0x4B47
 
 /* From <linux/keyboard.h> */
-static const int NR_KEYS = 128;
-static const int MAX_NR_KEYMAPS = 256;
+#define NR_KEYS         128
+#define MAX_NR_KEYMAPS  256
 
 int loadkmap_main(int argc, char **argv)
 {
 	struct kbentry ke;
-	u_short *ibuff;
-	int i, j, fd, readsz, pos, ibuffsz = NR_KEYS * sizeof(u_short);
-	char flags[MAX_NR_KEYMAPS], buff[7];
+	int i, j, fd;
+	u_short ibuff[NR_KEYS];
+	char flags[MAX_NR_KEYMAPS];
+	char buff[7];
 
 	if (argc != 1)
 		bb_show_usage();
 
-	fd = open(CURRENT_VC, O_RDWR);
-	if (fd < 0)
-		bb_perror_msg_and_die("Error opening " CURRENT_VC);
+	fd = bb_xopen(CURRENT_VC, O_RDWR);
 
-	read(0, buff, 7);
-	if (0 != strncmp(buff, BINARY_KEYMAP_MAGIC, 7))
+	if ((bb_full_read(0, buff, 7) != 7) || (strncmp(buff, BINARY_KEYMAP_MAGIC, 7) != 0))
 		bb_error_msg_and_die("This is not a valid binary keymap.");
 
-	if (MAX_NR_KEYMAPS != read(0, flags, MAX_NR_KEYMAPS))
+	if (bb_full_read(0, flags, MAX_NR_KEYMAPS) != MAX_NR_KEYMAPS)
 		bb_perror_msg_and_die("Error reading keymap flags");
 
-	ibuff = (u_short *) xmalloc(ibuffsz);
-
 	for (i = 0; i < MAX_NR_KEYMAPS; i++) {
 		if (flags[i] == 1) {
-			pos = 0;
-			while (pos < ibuffsz) {
-				if ((readsz = read(0, (char *) ibuff + pos, ibuffsz - pos)) < 0)
-					bb_perror_msg_and_die("Error reading keymap");
-				pos += readsz;
-			}
+			bb_full_read(0, ibuff, NR_KEYS * sizeof(u_short));
 			for (j = 0; j < NR_KEYS; j++) {
 				ke.kb_index = j;
 				ke.kb_table = i;
@@ -82,6 +74,7 @@
 			}
 		}
 	}
+
 	/* Don't bother to close files.  Exit does that 
 	 * automagically, so we can save a few bytes */
 	/* close(fd); */




More information about the busybox-cvs mailing list