From e3821380524f1fa75a6e38a3d1dad728ff3b48e8 Mon Sep 17 00:00:00 2001 From: Oswald Buddenhagen Date: Fri, 24 Aug 2012 14:49:13 +0200 Subject: [PATCH] fix ProString::mid() without second parameter we need to interpret the -1 as unsigned for it to mean "unlimited". Change-Id: I130b809749fa1679a5b2444286baf48cc095c7bf Reviewed-by: Daniel Teske --- src/shared/proparser/proitems.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/shared/proparser/proitems.cpp b/src/shared/proparser/proitems.cpp index e51e724eee5..962268ca7da 100644 --- a/src/shared/proparser/proitems.cpp +++ b/src/shared/proparser/proitems.cpp @@ -299,7 +299,7 @@ ProString ProString::mid(int off, int len) const off = m_length; ret.m_offset += off; ret.m_length -= off; - if (ret.m_length > len) + if ((uint)ret.m_length > (uint)len) // Unsigned comparison to interpret < 0 as infinite ret.m_length = len; return ret; }