summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Kientzle <kientzle@acm.org>2016-02-21 11:43:03 -0800
committerTim Kientzle <kientzle@acm.org>2016-02-21 11:43:03 -0800
commit37649d274867edd2dd25d8a3057c3b6cd81ce83e (patch)
treea8be5c557cff59ece6d964d64c74c29a19833054
parent41a759a7cc0e68814db3b88db675a741c2199f14 (diff)
downloadlibarchive-37649d274867edd2dd25d8a3057c3b6cd81ce83e.tar.gz
Clarify the limit on number of filters
-rw-r--r--libarchive/archive_read.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/libarchive/archive_read.c b/libarchive/archive_read.c
index 033ed8b5..0bbacc8f 100644
--- a/libarchive/archive_read.c
+++ b/libarchive/archive_read.c
@@ -547,16 +547,20 @@ archive_read_open1(struct archive *_a)
* it wants to handle this stream. Repeat until we've finished
* building the pipeline.
*/
+
+/* We won't build a filter pipeline with more stages than this. */
+#define MAX_NUMBER_FILTERS 25
+
static int
choose_filters(struct archive_read *a)
{
- int number_bidders, i, bid, best_bid, n;
+ int number_bidders, i, bid, best_bid, number_filters;
struct archive_read_filter_bidder *bidder, *best_bidder;
struct archive_read_filter *filter;
ssize_t avail;
int r;
- for (n = 0; n < 25; ++n) {
+ for (number_filters = 0; number_filters < MAX_NUMBER_FILTERS; ++number_filters) {
number_bidders = sizeof(a->bidders) / sizeof(a->bidders[0]);
best_bid = 0;