summaryrefslogtreecommitdiff
path: root/archival/libunarchive
diff options
context:
space:
mode:
authorDenis Vlasenko2006-09-03 14:04:33 +0000
committerDenis Vlasenko2006-09-03 14:04:33 +0000
commit314908da9e156e3100275598dde6848da038d65d (patch)
treed12f6ab92f2435ac599eb1f45b7bfeb8d7aafc1f /archival/libunarchive
parent65113e8d97c62b6aed560ac73961c8a9020c1568 (diff)
downloadbusybox-314908da9e156e3100275598dde6848da038d65d.zip
busybox-314908da9e156e3100275598dde6848da038d65d.tar.gz
tar: revert older fix (non-portable), added new one.
testsuite tar-extracts-all-subdirs now passes.
Diffstat (limited to 'archival/libunarchive')
-rw-r--r--archival/libunarchive/filter_accept_reject_list.c4
-rw-r--r--archival/libunarchive/find_list_entry.c40
2 files changed, 38 insertions, 6 deletions
diff --git a/archival/libunarchive/filter_accept_reject_list.c b/archival/libunarchive/filter_accept_reject_list.c
index be56a44..a3a938c 100644
--- a/archival/libunarchive/filter_accept_reject_list.c
+++ b/archival/libunarchive/filter_accept_reject_list.c
@@ -14,14 +14,14 @@
char filter_accept_reject_list(archive_handle_t *archive_handle)
{
const char *key = archive_handle->file_header->name;
- const llist_t *reject_entry = find_list_entry(archive_handle->reject, key);
+ const llist_t *reject_entry = find_list_entry2(archive_handle->reject, key);
const llist_t *accept_entry;
/* If the key is in a reject list fail */
if (reject_entry) {
return(EXIT_FAILURE);
}
- accept_entry = find_list_entry(archive_handle->accept, key);
+ accept_entry = find_list_entry2(archive_handle->accept, key);
/* Fail if an accept list was specified and the key wasnt in there */
if ((accept_entry == NULL) && archive_handle->accept) {
diff --git a/archival/libunarchive/find_list_entry.c b/archival/libunarchive/find_list_entry.c
index 57ffec6..d1afc72 100644
--- a/archival/libunarchive/find_list_entry.c
+++ b/archival/libunarchive/find_list_entry.c
@@ -9,14 +9,46 @@
#include <stdlib.h>
#include "unarchive.h"
-/* Find a string in a list */
+/* Find a string in a shell pattern list */
const llist_t *find_list_entry(const llist_t *list, const char *filename)
{
while (list) {
- if (fnmatch(list->data, filename, FNM_LEADING_DIR) == 0) {
- return (list);
+ if (fnmatch(list->data, filename, 0) == 0) {
+ return list;
}
list = list->link;
}
- return(NULL);
+ return NULL;
+}
+
+/* Same, but compares only path components present in pattern
+ * (extra trailing path components in filename are assumed to match)
+ */
+const llist_t *find_list_entry2(const llist_t *list, const char *filename)
+{
+ char buf[PATH_MAX];
+ int pattern_slash_cnt;
+ const char *c;
+ char *d;
+
+ while (list) {
+ c = list->data;
+ pattern_slash_cnt = 0;
+ while (*c)
+ if (*c++ == '/') pattern_slash_cnt++;
+ c = filename;
+ d = buf;
+ /* paranoia is better that buffer overflows */
+ while (*c && d != buf + sizeof(buf)-1) {
+ if (*c == '/' && --pattern_slash_cnt < 0)
+ break;
+ *d++ = *c++;
+ }
+ *d = '\0';
+ if (fnmatch(list->data, buf, 0) == 0) {
+ return list;
+ }
+ list = list->link;
+ }
+ return NULL;
}