diff options
| author | 2013-01-24 01:48:23 +0100 | |
|---|---|---|
| committer | 2013-01-24 01:48:23 +0100 | |
| commit | cce3cca8d31e0f7a9e8609b02bab6ab93eac0fb3 (patch) | |
| tree | 6c24c5d86d2434305b2c599f0015be013983ab27 | |
| parent | d61d30ab8edafb8cbac3eaa1935abdd2257eeaee (diff) | |
| download | libimobiledevice-cce3cca8d31e0f7a9e8609b02bab6ab93eac0fb3.tar.gz libimobiledevice-cce3cca8d31e0f7a9e8609b02bab6ab93eac0fb3.tar.bz2 | |
idevicebackup2: make sure -1 is returned if invalid arguments are passed
| -rw-r--r-- | tools/idevicebackup2.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tools/idevicebackup2.c b/tools/idevicebackup2.c index bcaad5c..b850329 100644 --- a/tools/idevicebackup2.c +++ b/tools/idevicebackup2.c | |||
| @@ -1312,7 +1312,7 @@ int main(int argc, char *argv[]) | |||
| 1312 | i++; | 1312 | i++; |
| 1313 | if (!argv[i] || (strlen(argv[i]) != 40)) { | 1313 | if (!argv[i] || (strlen(argv[i]) != 40)) { |
| 1314 | print_usage(argc, argv); | 1314 | print_usage(argc, argv); |
| 1315 | return 0; | 1315 | return -1; |
| 1316 | } | 1316 | } |
| 1317 | udid = strdup(argv[i]); | 1317 | udid = strdup(argv[i]); |
| 1318 | continue; | 1318 | continue; |
| @@ -1321,7 +1321,7 @@ int main(int argc, char *argv[]) | |||
| 1321 | i++; | 1321 | i++; |
| 1322 | if (!argv[i] || (strlen(argv[i]) != 40)) { | 1322 | if (!argv[i] || (strlen(argv[i]) != 40)) { |
| 1323 | print_usage(argc, argv); | 1323 | print_usage(argc, argv); |
| 1324 | return 0; | 1324 | return -1; |
| 1325 | } | 1325 | } |
| 1326 | source_udid = strdup(argv[i]); | 1326 | source_udid = strdup(argv[i]); |
| 1327 | continue; | 1327 | continue; |
| @@ -1355,7 +1355,7 @@ int main(int argc, char *argv[]) | |||
| 1355 | i++; | 1355 | i++; |
| 1356 | if (!argv[i]) { | 1356 | if (!argv[i]) { |
| 1357 | print_usage(argc, argv); | 1357 | print_usage(argc, argv); |
| 1358 | return 0; | 1358 | return -1; |
| 1359 | } | 1359 | } |
| 1360 | if (backup_password) | 1360 | if (backup_password) |
| 1361 | free(backup_password); | 1361 | free(backup_password); |
| @@ -1379,7 +1379,7 @@ int main(int argc, char *argv[]) | |||
| 1379 | if (!argv[i]) { | 1379 | if (!argv[i]) { |
| 1380 | printf("No argument given for encryption command; requires either 'on' or 'off'.\n"); | 1380 | printf("No argument given for encryption command; requires either 'on' or 'off'.\n"); |
| 1381 | print_usage(argc, argv); | 1381 | print_usage(argc, argv); |
| 1382 | return 0; | 1382 | return -1; |
| 1383 | } | 1383 | } |
| 1384 | if (!strcmp(argv[i], "on")) { | 1384 | if (!strcmp(argv[i], "on")) { |
| 1385 | cmd_flags |= CMD_FLAG_ENCRYPTION_ENABLE; | 1385 | cmd_flags |= CMD_FLAG_ENCRYPTION_ENABLE; |
| @@ -1426,7 +1426,7 @@ int main(int argc, char *argv[]) | |||
| 1426 | if (!argv[i]) { | 1426 | if (!argv[i]) { |
| 1427 | printf("Old and new passwords have to be passed as arguments for the changepw command\n"); | 1427 | printf("Old and new passwords have to be passed as arguments for the changepw command\n"); |
| 1428 | print_usage(argc, argv); | 1428 | print_usage(argc, argv); |
| 1429 | return 0; | 1429 | return -1; |
| 1430 | } | 1430 | } |
| 1431 | newpw = strdup(argv[i]); | 1431 | newpw = strdup(argv[i]); |
| 1432 | } | 1432 | } |
| @@ -1437,7 +1437,7 @@ int main(int argc, char *argv[]) | |||
| 1437 | } | 1437 | } |
| 1438 | else { | 1438 | else { |
| 1439 | print_usage(argc, argv); | 1439 | print_usage(argc, argv); |
| 1440 | return 0; | 1440 | return -1; |
| 1441 | } | 1441 | } |
| 1442 | } | 1442 | } |
| 1443 | 1443 | ||
