cp,mv: fix -t DIR option

When the '-t DIR' option is used the loop over the remaining
arguments should terminate when a NULL pointer is reached.

function                                             old     new   delta
mv_main                                              585     590      +5
cp_main                                              492     496      +4
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 2/0 up/down: 9/0)                 Total: 9 bytes

Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

2 files changed, 4 insertions, 4 deletions

diff --git a/coreutils/cp.c b/coreutils/cp.c
index b7f0e290f..50ca1ccea 100644
--- a/coreutils/cp.c
+++ b/coreutils/cp.c
@@ -252,7 +252,7 @@ int cp_main(int argc, char **argv)
                         }
                 }
         }
-        /* else: last is DIR from "t -DIR" */
+        /* else: last is DIR from "-t DIR" */
 
         while (1) {
 #if ENABLE_FEATURE_CP_LONG_OPTIONS
@@ -274,7 +274,7 @@ int cp_main(int argc, char **argv)
                 if (copy_file(*argv, dest, flags) < 0) {
                         status = EXIT_FAILURE;
                 }
-                if (*++argv == last) {
+                if (!*++argv || *argv == last) {
                         /* possibly leaking dest... */
                         break;
                 }
diff --git a/coreutils/mv.c b/coreutils/mv.c
index eae2e99bb..fd2422683 100644
--- a/coreutils/mv.c
+++ b/coreutils/mv.c
@@ -108,7 +108,7 @@ int mv_main(int argc, char **argv)
                         /* else: fall through into "do { move SRC to DIR/SRC } while" loop */
                 }
         }
-        /* else: last is DIR from "t -DIR" */
+        /* else: last is DIR from "-t DIR" */
 
         do {
                 dest = concat_path_file(last, bb_get_last_path_component_strip(*argv));
@@ -185,7 +185,7 @@ int mv_main(int argc, char **argv)
                 if (dest != last) {
                         free((void *) dest);
                 }
-        } while (*++argv != last);
+        } while (*++argv && *argv != last);
 
         return status;
 }