commit 7c439b563825802c7b968b52699f03e786ad0b75
author Anton Bikineev <bikineev@chromium.org> Mon Apr 05 17:20:15 2021 +0000
committer Chromium commit bot <commit-bot@chromium.org> Mon Apr 05 17:20:15 2021 +0000
tree 4b48c32f74d21a3ea576d87ec6b5b839b7b82591
parent e60430d1601c75a3017ca26b4b61c78a439115f5

Remove tautological 'unsigned expr < 0' comparison

This is the result of compiling Chromium with
-Wtautological-unsigned-zero-compare. For more details, see:
https://chromium-review.googlesource.com/c/chromium/src/+/2802412

Change-Id: I792e98c3ed518c96d6f5009c49bb81628291979e
Bug: chromium:1195670
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/78970
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Auto-Submit: Anton Bikineev <bikineev@chromium.org>

fpdfsdk/fpdf_ppo.cpp

diff --git a/fpdfsdk/fpdf_ppo.cpp b/fpdfsdk/fpdf_ppo.cpp
index 489d610..5c478cb 100644
--- a/fpdfsdk/fpdf_ppo.cpp
+++ b/fpdfsdk/fpdf_ppo.cpp
@@ -239,8 +239,7 @@
 
       uint32_t nEndPageNum = pdfium::base::checked_cast<uint32_t>(
           atoi(cbMidRange.Substr(nMid, nEnd).c_str()));
-      if (nStartPageNum < 0 || nStartPageNum > nEndPageNum ||
-          nEndPageNum > nCount) {
+      if (nStartPageNum > nEndPageNum || nEndPageNum > nCount) {
         return false;
       }
       for (uint32_t i = nStartPageNum; i <= nEndPageNum; ++i) {