From 7851f46fbb1e84fe4fa62a36d1f79b936f040e66 Mon Sep 17 00:00:00 2001 From: Lars Henriksen Date: Sun, 18 Nov 2018 18:32:11 +0100 Subject: CLI: filter options have no effect for dates before the epoch With the exception of filter.type_mask, a filter is only applied if set explicitly on the command line with a filter option. Whether that is the case, is determined by comparison with the initialization value. For date related filters (start_from/to, end_from/to) that is -1, hence the criterion is != -1, not >= 0. In generel, a filter initialization value should be invalid (i.e. one that cannot be set explicitly). As times before the epoch (1 January 1970 00:00:00 UTC) are negative, -1 is a valid Unix time. However, as it cannot be set from the command line, it is probably no problem? Signed-off-by: Lars Henriksen Signed-off-by: Lukas Fleischer --- src/apoint.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/apoint.c') diff --git a/src/apoint.c b/src/apoint.c index d200cf3..e8f9ee8 100644 --- a/src/apoint.c +++ b/src/apoint.c @@ -234,13 +234,13 @@ struct apoint *apoint_scan(FILE * f, struct tm start, struct tm end, return NULL; if (filter->regex && regexec(filter->regex, buf, 0, 0, 0)) return NULL; - if (filter->start_from >= 0 && tstart < filter->start_from) + if (filter->start_from != -1 && tstart < filter->start_from) return NULL; - if (filter->start_to >= 0 && tstart > filter->start_to) + if (filter->start_to != -1 && tstart > filter->start_to) return NULL; - if (filter->end_from >= 0 && tend < filter->end_from) + if (filter->end_from != -1 && tend < filter->end_from) return NULL; - if (filter->end_to >= 0 && tend > filter->end_to) + if (filter->end_to != -1 && tend > filter->end_to) return NULL; } -- cgit v1.2.3-54-g00ecf