From dd539f76877000e333796da8f30ea760c080a69a Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Wed, 1 Nov 2006 20:20:37 +0000 Subject: [PATCH] Unneeded code removed, usused field "unsigned pscpu" removed --- include/libbb.h | 7 +++---- libbb/procps.c | 8 ++++---- procps/kill.c | 7 ++++--- procps/ps.c | 4 ++-- util-linux/hwclock.c | 2 +- 5 files changed, 14 insertions(+), 14 deletions(-) diff --git a/include/libbb.h b/include/libbb.h index 343a93290..3ef1af1ab 100644 --- a/include/libbb.h +++ b/include/libbb.h @@ -501,13 +501,12 @@ void reset_ino_dev_hashtable(void); #endif #endif typedef struct { - pid_t pid, ppid; + int pid, ppid; char user[9]; char state[4]; unsigned long rss; #ifdef CONFIG_FEATURE_TOP_CPU_USAGE_PERCENTAGE unsigned pcpu; - unsigned pscpu; unsigned long stime, utime; #endif char *cmd; @@ -525,7 +524,7 @@ extern const char bb_uuenc_tbl_base64[]; extern const char bb_uuenc_tbl_std[]; void bb_uuencode(const unsigned char *s, char *store, const int length, const char *tbl); -typedef struct _sha1_ctx_t_ { +typedef struct sha1_ctx_t { uint32_t count[2]; uint32_t hash[5]; uint32_t wbuf[16]; @@ -534,7 +533,7 @@ void sha1_begin(sha1_ctx_t *ctx); void sha1_hash(const void *data, size_t length, sha1_ctx_t *ctx); void *sha1_end(void *resbuf, sha1_ctx_t *ctx); -typedef struct _md5_ctx_t_ { +typedef struct md5_ctx_t { uint32_t A; uint32_t B; uint32_t C; diff --git a/libbb/procps.c b/libbb/procps.c index 15a1cf74b..9d73fc4b7 100644 --- a/libbb/procps.c +++ b/libbb/procps.c @@ -36,13 +36,13 @@ procps_status_t * procps_scan(int save_user_arg0) struct dirent *entry; char *name; - int n; - char status[32]; - char *status_tail; char buf[PROCPS_BUFSIZE]; + char status[sizeof("/proc//cmdline") + sizeof(int)*3]; + char *status_tail; procps_status_t curstatus; - int pid; long tasknice; + int pid; + int n; struct stat sb; if (!dir) { diff --git a/procps/kill.c b/procps/kill.c index f22bdbe46..1b31a7291 100644 --- a/procps/kill.c +++ b/procps/kill.c @@ -84,9 +84,10 @@ do_it_now: pid_t sid; procps_status_t* p; - /* kill(-1, sig) on Linux (at least 2.1.x) - * might send signal to the calling process too */ - signal(SIGTERM, SIG_IGN); +// Cannot happen anyway? We don't TERM ourself, we STOP +// /* kill(-1, sig) on Linux (at least 2.1.x) +// * might send signal to the calling process too */ +// signal(SIGTERM, SIG_IGN); /* Now stop all processes */ kill(-1, SIGSTOP); /* Find out our own session id */ diff --git a/procps/ps.c b/procps/ps.c index da8c2a346..92608b114 100644 --- a/procps/ps.c +++ b/procps/ps.c @@ -25,7 +25,7 @@ int ps_main(int argc, char **argv) #if ENABLE_FEATURE_PS_WIDE || ENABLE_SELINUX #if ENABLE_FEATURE_PS_WIDE opt_complementary = "-:ww"; - USE_SELINUX(i =) getopt32(argc, argv, "w" USE_SELINUX("c"), &w_count); + USE_SELINUX(i =) getopt32(argc, argv, USE_SELINUX("c") "w", &w_count); /* if w is given once, GNU ps sets the width to 132, * if w is given more than once, it is "unlimited" */ @@ -40,7 +40,7 @@ int ps_main(int argc, char **argv) i = getopt32(argc, argv, "c"); #endif #if ENABLE_SELINUX - if ((i & (1+ENABLE_FEATURE_PS_WIDE)) && is_selinux_enabled()) + if ((i & 1) && is_selinux_enabled()) use_selinux = 1; #endif #endif /* ENABLE_FEATURE_PS_WIDE || ENABLE_SELINUX */ diff --git a/util-linux/hwclock.c b/util-linux/hwclock.c index 3a087725e..8fcd8c99c 100644 --- a/util-linux/hwclock.c +++ b/util-linux/hwclock.c @@ -106,7 +106,7 @@ static int show_clock(int utc) if (buffer[0]) buffer[strlen(buffer) - 1] = 0; - //printf("%s %.6f seconds %s\n", buffer, 0.0, utc ? "" : (ptm->tm_isdst ? tzname [1] : tzname [0])); + //printf("%s %.6f seconds %s\n", buffer, 0.0, utc ? "" : (ptm->tm_isdst ? tzname[1] : tzname[0])); printf( "%s %.6f seconds\n", buffer, 0.0); RELEASE_CONFIG_BUFFER(buffer);