enum backup_type get_version ();
/* FIXME: We should use a conversion routine that does reasonable error
- checking -- atoi doesn't. For now, punt. */
-#define intconv atoi
+ checking -- atol doesn't. For now, punt. */
+#define intconv atol
time_t get_date ();
/* Miscellaneous. */
/*------------------------------------------------------------------------.
-| Check if STRING is the decimal representation of number, and return its |
-| value. If not a decimal number, return -1. |
+| Check if STRING0 is the decimal representation of number, and store its |
+| value. If not a decimal number, return 0. |
`------------------------------------------------------------------------*/
static int
-check_decimal (const char *string)
+check_decimal (const char *string0, uintmax_t *result)
{
- int value = -1;
+ const char *string = string0;
+ uintmax_t value = 0;
- while (*string)
+ do
switch (*string)
{
case '0':
case '7':
case '8':
case '9':
- value = value < 0 ? *string - '0' : 10 * value + *string - '0';
- string++;
+ {
+ uintmax_t v10 = value * 10;
+ uintmax_t v10d = v10 + (*string - '0');
+ if (v10 / 10 != value || v10d < v10)
+ return 0;
+ value = v10d;
+ }
break;
default:
- return -1;
+ return 0;
}
- return value;
+ while (*++string);
+
+ *result = value;
+ return 1;
}
/*----------------------------------------------.
{"dereference", no_argument, NULL, 'h'},
{"diff", no_argument, NULL, 'd'},
{"directory", required_argument, NULL, 'C'},
+ {"ending-file", required_argument, NULL, 'E'},
{"exclude", required_argument, NULL, EXCLUDE_OPTION},
{"exclude-from", required_argument, NULL, 'X'},
{"extract", no_argument, NULL, 'x'},
-h, --dereference dump instead the files symlinks point to\n\
--no-recursion avoid descending automatically in directories\n\
-l, --one-file-system stay in local file system when creating archive\n\
+ -E, --ending-file=NAME end reading the archive before file NAME\n\
-K, --starting-file=NAME begin at file NAME in the archive\n"),
stdout);
#if !MSDOS
| Parse the options for tar. |
`----------------------------*/
-/* Available option letters are DEHIJQY and aejnqy. Some are reserved:
+/* Available option letters are DHIJQY and aejnqy. Some are reserved:
y per-file gzip compression
Y per-block gzip compression */
#define OPTION_STRING \
- "-01234567ABC:F:GK:L:MN:OPRST:UV:WX:Zb:cdf:g:hiklmoprstuvwxz"
+ "-01234567ABC:E:F:GK:L:MN:OPRST:UV:WX:Zb:cdf:g:hiklmoprstuvwxz"
static void
set_subcommand_option (enum subcommand subcommand)
case 'b':
blocking_factor = intconv (optarg);
- record_size = blocking_factor * BLOCKSIZE;
+ record_size = blocking_factor * (size_t) BLOCKSIZE;
break;
case OBSOLETE_READ_FULL_RECORDS:
set_subcommand_option (DIFF_SUBCOMMAND);
break;
+ case 'E':
+ ending_file_option = optarg;
+ break;
+
case 'f':
if (archive_names == allocated_archive_names)
{
case GROUP_OPTION:
if (!gname_to_gid (optarg, &group_option))
- if (!check_decimal (optarg) >= 0)
- ERROR ((TAREXIT_FAILURE, 0, _("Invalid group given on option")));
- else
- group_option = check_decimal (optarg);
+ {
+ uintmax_t g;
+ if (!check_decimal (optarg, &g) || g != (gid_t) g)
+ ERROR ((TAREXIT_FAILURE, 0, _("Invalid group given on option")));
+ else
+ group_option = g;
+ }
break;
case MODE_OPTION:
case OWNER_OPTION:
if (!uname_to_uid (optarg, &owner_option))
- if (!check_decimal (optarg) >= 0)
- ERROR ((TAREXIT_FAILURE, 0, _("Invalid owner given on option")));
- else
- owner_option = check_decimal (optarg);
+ {
+ uintmax_t u;
+ if (!check_decimal (optarg, &u) || u != (uid_t) u)
+ ERROR ((TAREXIT_FAILURE, 0, _("Invalid owner given on option")));
+ else
+ owner_option = u;
+ }
break;
case POSIX_OPTION: