From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48295) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bIwvL-0004Aa-W0 for qemu-devel@nongnu.org; Fri, 01 Jul 2016 07:47:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bIwvI-00068D-T7 for qemu-devel@nongnu.org; Fri, 01 Jul 2016 07:47:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42904) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bIwvI-00067z-NY for qemu-devel@nongnu.org; Fri, 01 Jul 2016 07:47:52 -0400 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 3ECE315447 for ; Fri, 1 Jul 2016 11:47:52 +0000 (UTC) From: Markus Armbruster Date: Fri, 1 Jul 2016 13:47:49 +0200 Message-Id: <1467373669-30915-5-git-send-email-armbru@redhat.com> In-Reply-To: <1467373669-30915-1-git-send-email-armbru@redhat.com> References: <1467373669-30915-1-git-send-email-armbru@redhat.com> Subject: [Qemu-devel] [PATCH v3 4/4] log: Permit -dfilter 0..0xffffffffffffffff List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: eblake@redhat.com, mst@redhat.com, pbonzini@redhat.com Works fine since the previous commit fixed the underlying range data type. Of course it filters out nothing, but so does 0..1,2..0xffffffffffffffff, and we don't bother rejecting that either. Signed-off-by: Markus Armbruster Reviewed-by: Eric Blake --- tests/test-logging.c | 5 +++-- util/log.c | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/test-logging.c b/tests/test-logging.c index b6fa94e..cdf13c6 100644 --- a/tests/test-logging.c +++ b/tests/test-logging.c @@ -73,8 +73,9 @@ static void test_parse_range(void) g_assert_false(qemu_log_in_addr_range(UINT64_MAX - 1)); qemu_set_dfilter_ranges("0..0xffffffffffffffff", &err); - error_free_or_abort(&err); - + g_assert(qemu_log_in_addr_range(0)); + g_assert(qemu_log_in_addr_range(UINT64_MAX)); + qemu_set_dfilter_ranges("2..1", &err); error_free_or_abort(&err); diff --git a/util/log.c b/util/log.c index 4da635c..b6c75b1 100644 --- a/util/log.c +++ b/util/log.c @@ -204,7 +204,7 @@ void qemu_set_dfilter_ranges(const char *filter_spec, Error **errp) default: g_assert_not_reached(); } - if (lob > upb || (lob == 0 && upb == UINT64_MAX)) { + if (lob > upb) { error_setg(errp, "Invalid range"); goto out; } -- 2.5.5