Applied patch from Larry Doolittle to remove some strlen calls, and add one

paranoia check to avoid  buffer underrun. Saves 120 text bytes.


git-svn-id: svn://busybox.net/trunk/busybox@2417 69ca8d6d-28ef-0310-b511-8ec308f3f277

1 files changed, 10 insertions, 7 deletions

diff --git a/util-linux/getopt.c b/util-linux/getopt.c
index b74dd65a1..95ecba6e6 100644
--- a/util-linux/getopt.c
+++ b/util-linux/getopt.c
@@ -243,20 +243,23 @@ void add_longopt(const char *name,int has_arg)
  */
 void add_long_options(char *options)
 {
-        int arg_opt;
+        int arg_opt, tlen;
         char *tokptr=strtok(options,", \t\n");
         while (tokptr) {
                 arg_opt=no_argument;
-                if (strlen(tokptr) > 0) {
-                        if (tokptr[strlen(tokptr)-1] == ':') {
-                                if (tokptr[strlen(tokptr)-2] == ':') {
-                                        tokptr[strlen(tokptr)-2]='\0';
+                tlen=strlen(tokptr);
+                if (tlen > 0) {
+                        if (tokptr[tlen-1] == ':') {
+                                if (tlen > 1 && tokptr[tlen-2] == ':') {
+                                        tokptr[tlen-2]='\0';
+                                        tlen -= 2;
                                         arg_opt=optional_argument;
                                 } else {
-                                        tokptr[strlen(tokptr)-1]='\0';
+                                        tokptr[tlen-1]='\0';
+                                        tlen -= 1;
                                         arg_opt=required_argument;
                                 }
-                                if (strlen(tokptr) == 0)
+                                if (tlen == 0)
                                         error_msg("empty long option after -l or --long argument");
                         }
                         add_longopt(tokptr,arg_opt);