Fixed a bunch of stuff:

	* Fixed segfault caused by "touch -c"
	* Fixed segfault caused by "rm -f"
	* Fixed segfault caused by "ln -s -s" and similar abuses.
	* Fixed segfault caused by "cp -a -a" and similar abuses.
	* Implemented "rm -- <foo>"
updated docs accordingly.
 -Erik

1 files changed, 28 insertions, 23 deletions

diff --git a/cp_mv.c b/cp_mv.c
index ac1e71b82..4445e7747 100644
--- a/cp_mv.c
+++ b/cp_mv.c
@@ -203,37 +203,42 @@ extern int cp_mv_main(int argc, char **argv)
 	if (dz_i == is_cp) {
 		recursiveFlag = preserveFlag = forceFlag = FALSE;
 		followLinks = TRUE;
-		while (**argv == '-') {
-			while (*++(*argv)) {
-				switch (**argv) {
-				case 'a':
-					followLinks = FALSE;
-					preserveFlag = TRUE;
-					recursiveFlag = TRUE;
-					break;
-				case 'd':
-					followLinks = FALSE;
-					break;
-				case 'p':
-					preserveFlag = TRUE;
-					break;
-				case 'R':
-					recursiveFlag = TRUE;
-					break;
-				case 'f':
-					forceFlag = TRUE;
-					break;
-				default:
-					usage(cp_mv_usage[is_cp]);
+		while (--argc >= 0 && *argv && **argv) {
+			while (**argv == '-') {
+				while (*++(*argv)) {
+					switch (**argv) {
+					case 'a':
+						followLinks = FALSE;
+						preserveFlag = TRUE;
+						recursiveFlag = TRUE;
+						break;
+					case 'd':
+						followLinks = FALSE;
+						break;
+					case 'p':
+						preserveFlag = TRUE;
+						break;
+					case 'R':
+						recursiveFlag = TRUE;
+						break;
+					case 'f':
+						forceFlag = TRUE;
+						break;
+					default:
+						usage(cp_mv_usage[is_cp]);
+					}
 				}
 			}
-			argc--;
 			argv++;
 		}
+		if (argc < 1) {
+			fatalError("cp: missing file argument\n");
+		}
 	} else {					/* (dz_i == is_mv) */
 		recursiveFlag = preserveFlag = TRUE;
 		followLinks = FALSE;
 	}
+	
 
 	if (strlen(argv[argc - 1]) > BUFSIZ) {
 		fprintf(stderr, name_too_long, "cp");