From e686719e381a4a256a3fefe8bff16e58c88a0cb7 Mon Sep 17 00:00:00 2001
From: Richard Hughes <richard@hughsie.com>
Date: Wed, 9 Nov 2016 19:30:35 +0000
Subject: [PATCH] dell: Fix a possibly-impossible-to-hit buffer overrun

Found using Coverity.
---
 src/fu-provider-dell.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/fu-provider-dell.c b/src/fu-provider-dell.c
index c0c09df74..40dff441f 100644
--- a/src/fu-provider-dell.c
+++ b/src/fu-provider-dell.c
@@ -503,7 +503,7 @@ fu_provider_dell_device_added_cb (GUsbContext *ctx,
 	parse_flags = fu_provider_dell_get_version_format ();
 
 	for (guint i = 0; i < dock_info->component_count; i++) {
-		if (i > MAX_COMPONENTS) {
+		if (i >= MAX_COMPONENTS) {
 			g_debug ("Dell: Too many components.  Invalid: #%u", i);
 			break;
 		}