POSIX states that getopt returns -1 when it is done parsing options,
authorTheodore Tso <tytso@mit.edu>
Wed, 6 May 1998 20:24:10 +0000 (20:24 +0000)
committerTheodore Tso <tytso@mit.edu>
Wed, 6 May 1998 20:24:10 +0000 (20:24 +0000)
not EOF.

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@10550 dc483132-0cff-0310-8789-dd5450dbe970

49 files changed:
src/appl/sample/sserver/ChangeLog
src/appl/sample/sserver/sserver.c
src/appl/simple/client/ChangeLog
src/appl/simple/client/sim_client.c
src/appl/simple/server/ChangeLog
src/appl/simple/server/sim_server.c
src/appl/telnet/libtelnet/ChangeLog
src/appl/telnet/libtelnet/getopt.c
src/clients/kdestroy/ChangeLog
src/clients/kdestroy/kdestroy.c
src/clients/kinit/ChangeLog
src/clients/kinit/kinit.c
src/clients/ksu/ChangeLog
src/clients/ksu/main.c
src/kadmin.v4/server/ChangeLog
src/kadmin.v4/server/admin_server.c
src/kadmin/dbutil/ChangeLog
src/kadmin/dbutil/kdb5_create.c
src/kadmin/dbutil/kdb5_destroy.c
src/kadmin/dbutil/kdb5_stash.c
src/kadmin/v5passwdd/ChangeLog
src/kadmin/v5passwdd/kpasswd.c
src/kdc/ChangeLog
src/kdc/main.c
src/krb524/ChangeLog
src/krb524/k524init.c
src/lib/kadm/ChangeLog
src/lib/kadm/t_dbentry.c
src/lib/kadm/t_ktentry.c
src/lib/kdb/ChangeLog
src/lib/kdb/t_kdb.c
src/lib/krb5/os/ChangeLog
src/lib/krb5/os/t_std_conf.c
src/prototype/ChangeLog [new file with mode: 0644]
src/prototype/getopt.c
src/tests/asn.1/ChangeLog
src/tests/asn.1/krb5_encode_test.c
src/tests/create/ChangeLog
src/tests/create/kdb5_mkdums.c
src/tests/hammer/ChangeLog
src/tests/hammer/kdc5_hammer.c
src/tests/verify/ChangeLog
src/tests/verify/kdb5_verify.c
src/util/db2/test/ChangeLog [new file with mode: 0644]
src/util/db2/test/btree.tests/ChangeLog [new file with mode: 0644]
src/util/db2/test/btree.tests/main.c
src/util/db2/test/dbtest.c
src/util/windows/ChangeLog
src/util/windows/getopt.c

index 36513524c2d7edb8b260168d47469e3884d46583..9973d04d62e21b0b81d9fcab9cc2e3ffc231ccc6 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * sserver.c (main): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:33:15 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index 78b291d36360a11c5c06c4d935828d806278e99f..e3b08962c09c3423223440f0b9637a799b311b62 100644 (file)
@@ -103,7 +103,7 @@ main(argc, argv)
      *  
      */
     opterr = 0;
-    while ((ch = getopt(argc, argv, "p:S:s:")) != EOF)
+    while ((ch = getopt(argc, argv, "p:S:s:")) != -1)
     switch (ch) {
     case 'p':
        port = atoi(optarg);
index 04d4e43f5b0189d5bb49155a594e8ee58e666084..3065caa8aa88d07461be508999dfa49a8061c7bf 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * sim_client.c (main): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:34:52 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index ec88a76b8e77f516e6c2aad5cff5940f5576be61..0838757c416a4d5c22b51e838f9709f08881ebdc 100644 (file)
@@ -108,7 +108,7 @@ main(argc, argv)
      *  
      */
     opterr = 0;
-    while ((ch = getopt(argc, argv, "p:m:h:s:")) != EOF)
+    while ((ch = getopt(argc, argv, "p:m:h:s:")) != -1)
     switch (ch) {
     case 'p':
        port = atoi(optarg);
index 3381a5b77004cbac533f9eb2db542c5148cd3600..68e9d4bae5c2ecc1808f90a16a8be8a9b3de2503 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * sim_server.c (argv): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:35:38 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index f57ec89110a686c8ddeed094021a9455dd6f05e6..7f1a824e46d1163cda3f0e03bf27189f8088409e 100644 (file)
@@ -98,7 +98,7 @@ char *argv[];
      *  
      */
     opterr = 0;
-    while ((ch = getopt(argc, argv, "p:s:S:")) != EOF)
+    while ((ch = getopt(argc, argv, "p:s:S:")) != -1)
     switch (ch) {
     case 'p':
        port = atoi(optarg);
index dd3448241aa525c990a40972fca1bacecc3c09b0..1a50bd4f034d6259694afda7a9c670c14f04f66e 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * getopt.c (getopt): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:37:20 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index a4a735684ca999574af80ed2ae838ab3c6481064..d61cc3c1fad4dcb166735a2930cd74f1544fd7e9 100644 (file)
@@ -67,12 +67,12 @@ getopt(nargc, nargv, ostr)
                optreset = 0;
                if (optind >= nargc || *(place = nargv[optind]) != '-') {
                        place = EMSG;
-                       return(EOF);
+                       return(-1);
                }
                if (place[1] && *++place == '-') {      /* found "--" */
                        ++optind;
                        place = EMSG;
-                       return(EOF);
+                       return(-1);
                }
        }                                       /* option letter okay? */
        if ((optopt = (int)*place++) == (int)':' ||
@@ -82,7 +82,7 @@ getopt(nargc, nargv, ostr)
                 * assume it means EOF.
                 */
                if (optopt == (int)'-')
-                       return(EOF);
+                       return(-1);
                if (!*place)
                        ++optind;
                if (opterr && *ostr != ':') {
index 15b38506d7fab27ac6e5c8f2378ec52fdfad8e1e..428f1d3faa525a87ad408779f22393e617219b13 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kdestroy.c (main): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:40:31 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index 97869a316b7fe5d7db1bbeb8d148d0df7bbe0529..b52ccb246dbb4ca7e39e99b4e0db651b1e76a48b 100644 (file)
@@ -55,7 +55,7 @@ main(argc, argv)
     if (strrchr(argv[0], '/'))
        argv[0] = strrchr(argv[0], '/')+1;
 
-    while ((c = getopt(argc, argv, "qc:")) != EOF) {
+    while ((c = getopt(argc, argv, "qc:")) != -1) {
        switch (c) {
        case 'q':
            quiet = 1;
index 3fadd99fa9534b5ae20eb090ea83537a729b80fd..65bcc031eba8b6b6b694d58cb7dd806fc840551c 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kinit.c (main): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:42:14 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index 6c3f037e60e730dda482814d1bfcc5d92ed47704..6f7b840c98981c3e5cd572c288f5f875e015975e 100644 (file)
@@ -105,7 +105,7 @@ main(argc, argv)
     if (strrchr(argv[0], '/'))
        argv[0] = strrchr(argv[0], '/')+1;
 
-    while ((option = getopt(argc, argv, "r:Rfpl:s:c:kt:vS:")) != EOF) {
+    while ((option = getopt(argc, argv, "r:Rfpl:s:c:kt:vS:")) != -1) {
        switch (option) {
        case 'r':
            options |= KDC_OPT_RENEWABLE;
index 0f8d4b13a7dbabd2ad5adb0b39fb2300cd603f0b..a7f563ac2de00d8621c2fd88f69ea032cf94b2ab 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * main.c (main): POSIX states that getopt returns -1 when it
+               is done parsing options, not EOF.
+
 Wed Feb 18 15:44:15 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index c5daf939a345ef583b0152b129082832b9fd745d..31a255cf3669821b680a330ce8c95967d4c9c63e 100644 (file)
@@ -172,7 +172,7 @@ char * dir_of_cc_source;
         }
 
 
-    while(!done && ((option = getopt(pargc, pargv,"n:c:C:r:a:zZDfpkql:e:")) != EOF)){
+    while(!done && ((option = getopt(pargc, pargv,"n:c:C:r:a:zZDfpkql:e:")) != -1)){
        switch (option) {
        case 'r':
            options.opt |= KDC_OPT_RENEWABLE;
index 431055f30f854ac43f4403fe80ab014132ab06fa..ba7dd7646ed4a275f530cca794c0b849b6d1d2ae 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * admin_server.c (argv): POSIX states that getopt returns -1 when
+               it is done parsing options, not EOF.
+
 Wed Feb 18 16:03:53 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index ac4032b38e6a3c0c3e4757425ee3c8acf044a43e..a1fd60c5ccee30d14df2026b6267ffbf1c3992ec 100644 (file)
@@ -102,7 +102,7 @@ char *argv[];
     memset(krbrlm, 0, sizeof(krbrlm));
 
     fascist_cpw = 1;           /* by default, enable fascist mode */
-    while ((c = getopt(argc, argv, "f:hnd:a:r:FN")) != EOF)
+    while ((c = getopt(argc, argv, "f:hnd:a:r:FN")) != -1)
        switch(c) {
        case 'f':                       /* Syslog file name change */
            prm.sysfile = optarg;
index 4f89fdd176dbc1ec0773f86f6064e41116eece99..3ab92e90e32f6aa39ef258e36049432d0e304ad5 100644 (file)
@@ -1,3 +1,10 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kdb5_stash.c (argv): 
+       * kdb5_destroy.c (kdb5_destroy): 
+       * kdb5_create.c (kdb5_create): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Fri Feb 27 23:32:38 1998  Theodore Ts'o  <tytso@rsts-11.mit.edu>
 
        * Makefile.in: Changed thisconfigdir to point at the kadmin
index c599228b493bf47bc3bdbd8a043bf3c1345a34c8..2e2c5f91262f90581ae74146f68b84a960040982 100644 (file)
@@ -138,7 +138,7 @@ void kdb5_create(argc, argv)
     if (strrchr(argv[0], '/'))
        argv[0] = strrchr(argv[0], '/')+1;
 
-    while ((optchar = getopt(argc, argv, "s")) != EOF) {
+    while ((optchar = getopt(argc, argv, "s")) != -1) {
        switch(optchar) {
        case 's':
            do_stash++;
index a0567c8b98b8ccccdc2f8b46e6861a6b62f274ee..c5f47ca2f46897ebfb11ba8c6641486ce71b0b32 100644 (file)
@@ -63,7 +63,7 @@ kdb5_destroy(argc, argv)
     dbname = global_params.dbname;
 
     optind = 1;
-    while ((optchar = getopt(argc, argv, "f")) != EOF) {
+    while ((optchar = getopt(argc, argv, "f")) != -1) {
        switch(optchar) {
        case 'f':
            force++;
index 9502c0952047a12bcf22402b1f9e76d2ebd77a1d..40084e2bb4af2c7be9112e8f37955ec2abea647d 100644 (file)
@@ -71,7 +71,7 @@ char *argv[];
     keyfile = global_params.stash_file;
 
     optind = 1;
-    while ((optchar = getopt(argc, argv, "f:")) != EOF) {
+    while ((optchar = getopt(argc, argv, "f:")) != -1) {
        switch(optchar) {
        case 'f':
            keyfile = optarg;
index 548e75e87bb726680ff7d10577e0a4e19dbb3fa5..90a19a8a217bc11687c105640635ad9ba9f9e1c6 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kpasswd.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Fri Feb 27 23:32:38 1998  Theodore Ts'o  <tytso@rsts-11.mit.edu>
 
        * Makefile.in: Changed thisconfigdir to point at the kadmin
index 1a7d1c49b81f5f89e944ea9c6df06179402699eb..e9785858b46b1200d82c393e144343a218dd49dd 100644 (file)
@@ -200,7 +200,7 @@ main(argc, argv)
      * Usage is:
      * kpasswd [-u user] [-l language]
      */
-    while ((option = getopt(argc, argv, kpwd_getoptstring)) != EOF) {
+    while ((option = getopt(argc, argv, kpwd_getoptstring)) != -1) {
        switch (option) {
        case 'u':
            if ((name = (char *) malloc(strlen(optarg)+1)) == NULL) {
index ce3fc0f27653e5f596d457e4defddbac79af35a6..614a095a4140f0cf0e181cb741ead51bf62ab5c8 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * main.c (initialize_realms): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Fri Mar 20 17:13:46 1998  Tom Yu  <tlyu@mit.edu>
 
        * kdc_util.c (add_to_transited): Check lengths, fix up comma
index 786312b9c0ed73f44f902d0c714305ff1ac7c4f1..d58ebc80ec9fa34d385cab0dbc8e774d3f0de762 100644 (file)
@@ -627,7 +627,7 @@ initialize_realms(kcontext, argc, argv)
      * Loop through the option list.  Each time we encounter a realm name,
      * use the previously scanned options to fill in for defaults.
      */
-    while ((c = getopt(argc, argv, "r:d:mM:k:R:e:p:s:n4:")) != EOF) {
+    while ((c = getopt(argc, argv, "r:d:mM:k:R:e:p:s:n4:")) != -1) {
        switch(c) {
        case 'r':                       /* realm name for db */
            if (!find_realm_data(optarg, (krb5_ui_4) strlen(optarg))) {
index c492ab3d8c893042d9316f18aa5458aa74751a1c..675070c2cf6732868db5a28139d52de268de203c 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * k524init.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Wed Feb 18 16:04:44 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in (thisconfigdir): Remove trailing slash.
index 3c355a29bf4361562746680b282339a92387ec7e..680991c4422dfe1a83fd49d0ef7ac8eb564a0250 100644 (file)
@@ -59,7 +59,7 @@ int main(argc, argv)
             exit(1);
      }
 
-     while(((option =  getopt(argc, argv, "p:n")) != EOF)) {
+     while(((option =  getopt(argc, argv, "p:n")) != -1)) {
         switch(option) {
           case 'p':
             princ = optarg;
index d35cd3a1a4b332e36750a9dafd535833b5806afb..0973153ac65f2beab8effe15e02969b34c0eff4f 100644 (file)
@@ -1,3 +1,9 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * t_ktentry.c (main): 
+       * t_dbentry.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Wed Feb 18 16:13:15 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in: Remove trailing slash from thisconfigdir.  Fix up
index 7fa5780194908727e41364c2b0cf53712ef17b69..461c8e7ef25456dae93d43d3768ccf836cf9ee63 100644 (file)
@@ -928,7 +928,7 @@ main(argc, argv)
 
     now = time((time_t *) NULL);
     SRAND((RAND_TYPE) now);
-    while ((option = getopt(argc, argv, "r:v")) != EOF) {
+    while ((option = getopt(argc, argv, "r:v")) != -1) {
        switch (option) {
        case 'r':
            if (sscanf(optarg, "%d", &randompasses) != 1) {
index 8242699d051f19cd6b2fb374b14e5a57f563e669..c59319cc66fa0d6cd4cf6da4ee77310d0e3cfe62 100644 (file)
@@ -366,7 +366,7 @@ main(argc, argv)
 
     now = time((time_t *) NULL);
     SRAND((RAND_TYPE) now);
-    while ((option = getopt(argc, argv, "r:v")) != EOF) {
+    while ((option = getopt(argc, argv, "r:v")) != -1) {
        switch (option) {
        case 'r':
            if (sscanf(optarg, "%d", &randompasses) != 1) {
index 6c692d1d1ba6c9616d38cbb08c58e240567f7a08..ec0b38457caa18bf972558d98f444af26d52fccf 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * t_kdb.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Wed Apr 15 18:06:34 1998  Tom Yu  <tlyu@mit.edu>
 
        * Makefile.in (SHLIB_EXPDEPS): 
index 75c0a373dc74489eb93dd247993e6f70eb676ce5..b14152602926870bee8a10ba681959e069887f51 100644 (file)
@@ -1015,7 +1015,7 @@ main(argc, argv)
     hash = 0;
 
     /* Parse argument list */
-    while ((option = getopt(argc, argv, "cd:n:prstvDh")) != EOF) {
+    while ((option = getopt(argc, argv, "cd:n:prstvDh")) != -1) {
        switch (option) {
        case 'c':
            check_cont = 1;
index 12619dd5d9b54f32771343bbf0a04832c17087b3..c9b7ece1f52b124e4f42dc0cf744cac199c08748 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * t_std_conf.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Fri Feb 27 18:03:33 1998  Theodore Ts'o  <tytso@rsts-11.mit.edu>
 
        * Makefile.in: Changed thisconfigdir to point at the lib/krb5
index 63a4bd7278d5571065b6aece0e2c2673ca2902e9..5e615102efb5e61812253671d076bfa426cf8c3a 100644 (file)
@@ -173,7 +173,7 @@ main(argc, argv)
                exit(1);
        }
 
-       while ((c = getopt(argc, argv, "dk:r:D:l:s:")) != EOF) {
+       while ((c = getopt(argc, argv, "dk:r:D:l:s:")) != -1) {
            switch (c) {
            case 'd': /* Get default realm */
                test_get_default_realm(ctx);
diff --git a/src/prototype/ChangeLog b/src/prototype/ChangeLog
new file mode 100644 (file)
index 0000000..e0f924c
--- /dev/null
@@ -0,0 +1,5 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * getopt.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
index 66cbad5d43dfaa0575d32aad7cd0856b540e244f..935f0bf5fadcc9ca8a46e9a52225204c6b232a84 100644 (file)
@@ -10,7 +10,7 @@ main(argc, argv)
     
     <<<other globals here>>>;
        
-    while ((c = getopt(argc, argv, "<<<>>>")) != EOF) {
+    while ((c = getopt(argc, argv, "<<<>>>")) != -1) {
        switch (c) {
            <<<add cases for arguments here>>>;
        case '?':
index 1e0a00736cc4a3a236397149a59b3df1f67af60f..83d6ca304a36ecddec99024b871d835b495b5b6b 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * krb5_encode_test.c (PRS): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Mon Mar 30 13:51:58 1998  Ezra Peisach  <epeisach@kangaroo.mit.edu>
 
        * ktest_equal.h: Add prototypes for ktest_equal_sam_challenge and 
index 95bdab6304721b8a62656c6ff090ba6f6e21358f..68fd441044eeb00d0ada805be767e6ee785229ed 100644 (file)
@@ -59,7 +59,7 @@ void PRS(argc, argv)
                print_constructed_length, print_skip_context,
                print_skip_tagnum, print_context_shortcut;
 
-       while ((optchar = getopt(argc, argv, "tp:")) != EOF) {
+       while ((optchar = getopt(argc, argv, "tp:")) != -1) {
                switch(optchar) {
                case 't':
                        do_trval = 1;
index 402ad04bb974f82ab5f3e41cbd2eed40bc7813ac..64e0b2801cf91ff135ae237be2542dc17258a75e 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kdb5_mkdums.c (argv): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Mon Mar 30 16:57:43 1998  Ezra Peisach  <epeisach@mit.edu>
 
        * kdb5_mkdums.c: Add parenthesis about assignements in conditionals.
index ca17b68153cfa50d3eb013b0a94020b0b632058c..18ee9e8ac67dadfc7f332cb23d94d54b02d60bc5 100644 (file)
@@ -119,7 +119,7 @@ char *argv[];
     num_to_create = 0;
     depth = 1;
 
-    while ((optchar = getopt(argc, argv, "D:P:p:n:d:r:k:M:e:m")) != EOF) {
+    while ((optchar = getopt(argc, argv, "D:P:p:n:d:r:k:M:e:m")) != -1) {
        switch(optchar) {
        case 'D':
            depth = atoi(optarg);       /* how deep to go */
index 8d652e76ee6b500086ac60b4982ab8e75669120f..59c5767eb21d4ffc5b2ae547a2fb8f9830e76bcc 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kdc5_hammer.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Mon Mar 30 16:48:17 1998  Ezra Peisach  <epeisach@kangaroo.mit.edu>
 
        * kdc5_hammer.c (usage): Change usage to refelect reality
index 9769f05418da208a7c7cfe2a0c711248cc430d77..54d567c430c47507b10c398b79fc3a7d0622c2dd 100644 (file)
@@ -145,7 +145,7 @@ main(argc, argv)
     errors = 0;
     enctypedone = 0;
 
-    while ((option = getopt(argc, argv, "D:p:n:c:R:k:P:e:bvr:t")) != EOF) {
+    while ((option = getopt(argc, argv, "D:p:n:c:R:k:P:e:bvr:t")) != -1) {
        switch (option) {
        case 't':
            do_timer = 1;
index bb9397a4483ca23ceb833820db82e4f611f9a1ff..12fb1f93b53fe38418ae865f83dd5394b6c75789 100644 (file)
@@ -1,3 +1,8 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * kdb5_verify.c (argv): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
 Mon Mar 30 16:46:20 1998  Ezra Peisach  <epeisach@kangaroo.mit.edu>
 
        * kdb5_verify.c: Declare main() as int instead of void.
index 5ef38d6d6d8d04977f0d176a81a8cdf548d65aa7..8a95d7125e7ccbe39328ef514ad3224a29ec6437 100644 (file)
@@ -119,7 +119,7 @@ char *argv[];
     num_to_check = 0;
     depth = 1;
 
-    while ((optchar = getopt(argc, argv, "D:P:p:n:d:r:R:k:M:e:m")) != EOF) {
+    while ((optchar = getopt(argc, argv, "D:P:p:n:d:r:R:k:M:e:m")) != -1) {
        switch(optchar) {
        case 'D':
            depth = atoi(optarg);       /* how deep to go */
diff --git a/src/util/db2/test/ChangeLog b/src/util/db2/test/ChangeLog
new file mode 100644 (file)
index 0000000..4e147d7
--- /dev/null
@@ -0,0 +1,5 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * dbtest.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
+
diff --git a/src/util/db2/test/btree.tests/ChangeLog b/src/util/db2/test/btree.tests/ChangeLog
new file mode 100644 (file)
index 0000000..cfd0b18
--- /dev/null
@@ -0,0 +1,4 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * main.c (main): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
index 61c9731cc9ab675ff4ff3c26ddb7c76e2ad3ab32..bbf1fcfacb8c164ca8862eb7d3c0fb7dc1474368 100644 (file)
@@ -135,7 +135,7 @@ main(argc, argv)
        b.prefix = NULL;
        b.lorder = 0;
 
-       while ((c = getopt(argc, argv, "bc:di:lp:ru")) != EOF) {
+       while ((c = getopt(argc, argv, "bc:di:lp:ru")) != -1) {
                switch (c) {
                case 'b':
                        b.lorder = BIG_ENDIAN;
index 170a0ea0f7003445d409cd170cebefe02a49e82d..f499047ae9cdbf4dbc5777a6c6d58414cd66af01 100644 (file)
@@ -101,7 +101,7 @@ main(argc, argv)
        fname = NULL;
        oflags = O_CREAT | O_RDWR | O_BINARY;
        sflag = 0;
-       while ((ch = getopt(argc, argv, "f:i:lo:s")) != EOF)
+       while ((ch = getopt(argc, argv, "f:i:lo:s")) != -1)
                switch (ch) {
                case 'f':
                        fname = optarg;
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..0484dd08a14a33d6be853adcaaf1ff7bbdffc15d 100644 (file)
@@ -0,0 +1,4 @@
+1998-05-06  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * getopt.c (getopt): POSIX states that getopt returns -1
+               when it is done parsing options, not EOF.
index ee38e72abe0f9d399066581af736994427f112a0..081520f55933f1f85d4ff306c689c9bfc4e8c497 100644 (file)
@@ -67,12 +67,12 @@ getopt(nargc, nargv, ostr)
                optreset = 0;
                if (optind >= nargc || *(place = nargv[optind]) != '-') {
                        place = EMSG;
-                       return(EOF);
+                       return(-1);
                }
                if (place[1] && *++place == '-') {      /* found "--" */
                        ++optind;
                        place = EMSG;
-                       return(EOF);
+                       return(-1);
                }
        }                                       /* option letter okay? */
        if ((optopt = (int)*place++) == (int)':' ||
@@ -82,7 +82,7 @@ getopt(nargc, nargv, ostr)
                 * assume it means EOF.
                 */
                if (optopt == (int)'-')
-                       return(EOF);
+                       return(-1);
                if (!*place)
                        ++optind;
                if (opterr && *ostr != ':') {