DOS-like: Treat \ and : as directory separators in addition to /.

Juan Manuel Guerrero had fixed this in his XZ Utils port
to DOS/DJGPP. The bug affects also Windows and OS/2.
This commit is contained in:
Lasse Collin 2010-12-12 14:50:04 +02:00
parent b7afd3e22a
commit e6baedddcf
1 changed files with 29 additions and 4 deletions

View File

@ -27,6 +27,30 @@ struct suffix_pair {
}; };
/// \brief Test if the char is a directory separator
static bool
is_dir_sep(char c)
{
#ifdef TUKLIB_DOSLIKE
return c == '/' || c == '\\' || c == ':';
#else
return c == '/';
#endif
}
/// \brief Test if the string contains a directory separator
static bool
has_dir_sep(const char *str)
{
#ifdef TUKLIB_DOSLIKE
return strpbrk(str, "/\\:") != NULL;
#else
return strchr(str, '/') != NULL;
#endif
}
/// \brief Checks if src_name has given compressed_suffix /// \brief Checks if src_name has given compressed_suffix
/// ///
/// \param suffix Filename suffix to look for /// \param suffix Filename suffix to look for
@ -44,7 +68,8 @@ test_suffix(const char *suffix, const char *src_name, size_t src_len)
// The filename must have at least one character in addition to // The filename must have at least one character in addition to
// the suffix. src_name may contain path to the filename, so we // the suffix. src_name may contain path to the filename, so we
// need to check for directory separator too. // need to check for directory separator too.
if (src_len <= suffix_len || src_name[src_len - suffix_len - 1] == '/') if (src_len <= suffix_len
|| is_dir_sep(src_name[src_len - suffix_len - 1]))
return 0; return 0;
if (strcmp(suffix, src_name + src_len - suffix_len) == 0) if (strcmp(suffix, src_name + src_len - suffix_len) == 0)
@ -199,9 +224,9 @@ suffix_get_dest_name(const char *src_name)
extern void extern void
suffix_set(const char *suffix) suffix_set(const char *suffix)
{ {
// Empty suffix and suffixes having a slash are rejected. Such // Empty suffix and suffixes having a directory separator are
// suffixes would break things later. // rejected. Such suffixes would break things later.
if (suffix[0] == '\0' || strchr(suffix, '/') != NULL) if (suffix[0] == '\0' || has_dir_sep(suffix))
message_fatal(_("%s: Invalid filename suffix"), optarg); message_fatal(_("%s: Invalid filename suffix"), optarg);
// Replace the old custom_suffix (if any) with the new suffix. // Replace the old custom_suffix (if any) with the new suffix.