svn commit: trunk/busybox: archival coreutils e2fsprogs include lib etc...

vodz at busybox.net vodz at busybox.net
Thu Sep 29 16:19:02 UTC 2005


Author: vodz
Date: 2005-09-29 09:18:57 -0700 (Thu, 29 Sep 2005)
New Revision: 11694

Log:
change interface to bb_xasprintf() - more perfect for me.
ln.c: error_msg(str)->error_msg(%s, str) - remove standart "feature" for hackers
reduce 100 bytes don't care in sum

Modified:
   trunk/busybox/archival/tar.c
   trunk/busybox/coreutils/expr.c
   trunk/busybox/coreutils/ln.c
   trunk/busybox/e2fsprogs/fsck.c
   trunk/busybox/e2fsprogs/mke2fs.c
   trunk/busybox/include/libbb.h
   trunk/busybox/libbb/bb_asprintf.c
   trunk/busybox/libbb/concat_path_file.c
   trunk/busybox/libbb/run_shell.c
   trunk/busybox/loginutils/adduser.c
   trunk/busybox/miscutils/less.c
   trunk/busybox/miscutils/mountpoint.c
   trunk/busybox/modutils/insmod.c
   trunk/busybox/networking/ifupdown.c
   trunk/busybox/procps/sysctl.c
   trunk/busybox/shell/cmdedit.c


Changeset:
Modified: trunk/busybox/archival/tar.c
===================================================================
--- trunk/busybox/archival/tar.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/archival/tar.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -695,13 +695,9 @@
 	}
 
 	/* Prepend '-' to the first argument if required */
-	if (argv[1][0] != '-') {
-		char *tmp;
+	if (argv[1][0] != '-') 
+		argv[1] = bb_xasprintf("-%s", argv[1]);
 
-		bb_xasprintf(&tmp, "-%s", argv[1]);
-		argv[1] = tmp;
-	}
-
 	/* Initialise default values */
 	tar_handle = init_handle();
 	tar_handle->flags = ARCHIVE_CREATE_LEADING_DIRS | ARCHIVE_PRESERVE_DATE | ARCHIVE_EXTRACT_UNCONDITIONAL;

Modified: trunk/busybox/coreutils/expr.c
===================================================================
--- trunk/busybox/coreutils/expr.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/coreutils/expr.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -147,7 +147,7 @@
 static void tostring (VALUE *v)
 {
 	if (v->type == integer) {
-               bb_xasprintf (&(v->u.s), "%d", v->u.i);
+		v->u.s = bb_xasprintf ("%d", v->u.i);
 		v->type = string;
 	}
 }

Modified: trunk/busybox/coreutils/ln.c
===================================================================
--- trunk/busybox/coreutils/ln.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/coreutils/ln.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -74,17 +74,17 @@
 			src_name = src;
 		}
 		if (!(flag & LN_SYMLINK) && stat(*argv, &statbuf)) {
-			bb_perror_msg(*argv);
+			bb_perror_msg("%s", *argv);
 			status = EXIT_FAILURE;
 			free(src_name);
 			continue;
 		}
 
 		if (flag & LN_BACKUP) {
-				char *backup = NULL;
-				bb_xasprintf(&backup, "%s%s", src, suffix);
+				char *backup;
+				backup = bb_xasprintf("%s%s", src, suffix);
 				if (rename(src, backup) < 0 && errno != ENOENT) {
-						bb_perror_msg(src);
+						bb_perror_msg("%s", src);
 						status = EXIT_FAILURE;
 						free(backup);
 						continue;

Modified: trunk/busybox/e2fsprogs/fsck.c
===================================================================
--- trunk/busybox/e2fsprogs/fsck.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/e2fsprogs/fsck.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -363,7 +363,6 @@
 {
   char *s;
   const char *tpl;
-  char *prog;
   char *p = string_copy(fsck_path);
   struct stat st;
 
@@ -371,12 +370,12 @@
   tpl = (strncmp(type, "fsck.", 5) ? "%s/fsck.%s" : "%s/%s");
 
   for(s = strtok(p, ":"); s; s = strtok(NULL, ":")) {
-	bb_xasprintf(&prog, tpl, s, type);
-	if (stat(prog, &st) == 0) break;
-	free(prog);
+	s = bb_xasprintf(tpl, s, type);
+	if (stat(s, &st) == 0) break;
+	free(s);
   }
   free(p);
-  return(s ? prog : NULL);
+  return(s);
 }
 
 static int progress_active(void)
@@ -410,7 +409,7 @@
 		return ENOMEM;
 	memset(inst, 0, sizeof(struct fsck_instance));
 
-	bb_xasprintf(&prog, "fsck.%s", type);
+	prog = bb_xasprintf("fsck.%s", type);
 	argv[0] = prog;
 	argc = 1;
 
@@ -1189,7 +1188,7 @@
 
 	/* Update our search path to include uncommon directories. */
 	if (oldpath) {
-		bb_xasprintf(&fsck_path, "%s:%s", fsck_prefix_path, oldpath);
+		fsck_path = bb_xasprintf("%s:%s", fsck_prefix_path, oldpath);
 	} else {
 		fsck_path = string_copy(fsck_prefix_path);
 	}

Modified: trunk/busybox/e2fsprogs/mke2fs.c
===================================================================
--- trunk/busybox/e2fsprogs/mke2fs.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/e2fsprogs/mke2fs.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -794,10 +794,7 @@
 
 	/* Update our PATH to include /sbin  */
 	if (oldpath) {
-		char *newpath;
-
-		bb_xasprintf(&newpath, "%s:%s", PATH_SET, oldpath);
-		putenv(newpath);
+		putenv (bb_xasprintf("%s:%s", PATH_SET, oldpath));
 	} else
 		putenv (PATH_SET);
 

Modified: trunk/busybox/include/libbb.h
===================================================================
--- trunk/busybox/include/libbb.h	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/include/libbb.h	2005-09-29 16:18:57 UTC (rev 11694)
@@ -408,9 +408,8 @@
 extern size_t bb_strlen(const char *string);
 #define strlen(x)   bb_strlen(x)
 
-void bb_xasprintf(char **string_ptr, const char *format, ...) __attribute__ ((format (printf, 2, 3)));
+char *bb_xasprintf(const char *format, ...) __attribute__ ((format (printf, 1, 2)));
 
-
 #define FAIL_DELAY    3
 extern void change_identity ( const struct passwd *pw );
 extern const char *change_identity_e2str ( const struct passwd *pw );

Modified: trunk/busybox/libbb/bb_asprintf.c
===================================================================
--- trunk/busybox/libbb/bb_asprintf.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/libbb/bb_asprintf.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -1,5 +1,5 @@
 /*
-   Copyright (C) 2002 Vladimir Oleynik <dzo at simtreas.ru>
+   Copyright (C) 2002,2005 Vladimir Oleynik <dzo at simtreas.ru>
 */
 
 #include <stdlib.h>
@@ -7,16 +7,18 @@
 #include <stdarg.h>
 #include "libbb.h"
 
-void bb_xasprintf(char **string_ptr, const char *format, ...)
+char *bb_xasprintf(const char *format, ...)
 {
 	va_list p;
 	int r;
+	char *string_ptr;
 
 	va_start(p, format);
-	r = vasprintf(string_ptr, format, p);
+	r = vasprintf(&string_ptr, format, p);
 	va_end(p);
 
 	if (r < 0) {
 		bb_perror_msg_and_die("bb_xasprintf");
 	}
+	return string_ptr;
 }

Modified: trunk/busybox/libbb/concat_path_file.c
===================================================================
--- trunk/busybox/libbb/concat_path_file.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/libbb/concat_path_file.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -30,7 +30,6 @@
 
 extern char *concat_path_file(const char *path, const char *filename)
 {
-	char *outbuf;
 	char *lc;
 
 	if (!path)
@@ -38,7 +37,5 @@
 	lc = last_char_is(path, '/');
 	while (*filename == '/')
 		filename++;
-	bb_xasprintf(&outbuf, "%s%s%s", path, (lc==NULL ? "/" : ""), filename);
-
-	return outbuf;
+	return bb_xasprintf("%s%s%s", path, (lc==NULL ? "/" : ""), filename);
 }

Modified: trunk/busybox/libbb/run_shell.c
===================================================================
--- trunk/busybox/libbb/run_shell.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/libbb/run_shell.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -84,11 +84,8 @@
 
 	args [0] = bb_get_last_path_component ( bb_xstrdup ( shell ));
 
-	if ( loginshell ) {
-		char *args0;
-		bb_xasprintf ( &args0, "-%s", args [0] );
-		args [0] = args0;
-	}
+	if ( loginshell ) 
+		args [0] = bb_xasprintf ("-%s", args [0]);
 
 	if ( command ) {
 		args [argno++] = "-c";

Modified: trunk/busybox/loginutils/adduser.c
===================================================================
--- trunk/busybox/loginutils/adduser.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/loginutils/adduser.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -117,7 +117,7 @@
 {
 	char *cmd;
 
-	bb_xasprintf(&cmd, "addgroup -g %d %s", gid, login);
+	cmd = bb_xasprintf("addgroup -g %d %s", gid, login);
 	system(cmd);
 	free(cmd);
 }

Modified: trunk/busybox/miscutils/less.c
===================================================================
--- trunk/busybox/miscutils/less.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/miscutils/less.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -208,7 +208,7 @@
 
 	for (i = 0; i <= num_flines; i++) {
 		safe_strncpy(current_line, flines[i], 256);
-		bb_xasprintf(&flines[i],"%5d %s", i + 1, current_line);
+		flines[i] = bb_xasprintf("%5d %s", i + 1, current_line);
 	}
 }
 
@@ -618,11 +618,8 @@
 
 static char *insert_highlights (char *line, int start, int end) {
 
-	char *new_line;
-
-	bb_xasprintf(&new_line, "%.*s%s%.*s%s%s", start, line, HIGHLIGHT,
+	return bb_xasprintf("%.*s%s%.*s%s%s", start, line, HIGHLIGHT,
 			end - start, line + start, NORMAL, line + end);
-	return new_line;
 }
 
 static char *process_regex_on_line(char *line, regex_t *pattern) {

Modified: trunk/busybox/miscutils/mountpoint.c
===================================================================
--- trunk/busybox/miscutils/mountpoint.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/miscutils/mountpoint.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -46,8 +46,8 @@
 			if (S_ISDIR(st.st_mode)) {
 				dev_t st_dev = st.st_dev;
 				ino_t st_ino = st.st_ino;
-				char *p;
-				bb_xasprintf(&p, "%s/..", arg);
+				char *p = bb_xasprintf("%s/..", arg);
+				
 				if (stat(p, &st) == 0) {
 					short ret = (st_dev != st.st_dev) ||
 						(st_dev == st.st_dev && st_ino == st.st_ino);

Modified: trunk/busybox/modutils/insmod.c
===================================================================
--- trunk/busybox/modutils/insmod.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/modutils/insmod.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -3756,10 +3756,10 @@
 
 #if defined(CONFIG_FEATURE_2_6_MODULES)
 	if (k_version > 4)
-		bb_xasprintf(&m_fullName, "%s.ko", tmp);
+		m_fullName = bb_xasprintf("%s.ko", tmp);
 	else
 #endif
-		bb_xasprintf(&m_fullName, "%s.o", tmp);
+		m_fullName = bb_xasprintf("%s.o", tmp);
 
 	if (!m_name) {
 		m_name = tmp;

Modified: trunk/busybox/networking/ifupdown.c
===================================================================
--- trunk/busybox/networking/ifupdown.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/networking/ifupdown.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -1010,7 +1010,7 @@
 		}
 	}
 
-	bb_xasprintf(&buf, "run-parts /etc/network/if-%s.d", opt);
+	buf = bb_xasprintf("run-parts /etc/network/if-%s.d", opt);
 	if ((*exec)(buf) != 1) {
 		return 0;
 	}

Modified: trunk/busybox/procps/sysctl.c
===================================================================
--- trunk/busybox/procps/sysctl.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/procps/sysctl.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -209,7 +209,7 @@
 		return -2;
 	}
 
-	bb_xasprintf(&tmpname, "%s%.*s", PROC_PATH, (equals - name), name);
+	tmpname = bb_xasprintf("%s%.*s", PROC_PATH, (equals - name), name);
 	outname = bb_xstrdup(tmpname + strlen(PROC_PATH));
 
 	while ((cptr = strchr(tmpname, '.')) != NULL)

Modified: trunk/busybox/shell/cmdedit.c
===================================================================
--- trunk/busybox/shell/cmdedit.c	2005-09-29 15:53:34 UTC (rev 11693)
+++ trunk/busybox/shell/cmdedit.c	2005-09-29 16:18:57 UTC (rev 11694)
@@ -621,7 +621,7 @@
 			/* Null usernames should result in all users as possible completions. */
 			if ( /*!userlen || */ !strncmp(ud, entry->pw_name, userlen)) {
 
-			       bb_xasprintf(&temp, "~%s/", entry->pw_name);
+				temp = bb_xasprintf("~%s/", entry->pw_name);
 				matches = xrealloc(matches, (nm + 1) * sizeof(char *));
 
 				matches[nm++] = temp;




More information about the busybox-cvs mailing list