tree ddfd5474969811c29e4ec1676518ed5dce42a289
parent bd37163c9ba3db96bfc185e892ad6ef82c7a8e9e
author Amit Nayar <amit@pspdfkit.com> 1586366049 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1586366049 +0000

Fix FindPageIndex() search algorithm.

Regression introduced in 5b4135d22ddd1450327931ee04fb3f14ada95471.

Required when reaching a leaf node of the Pages tree during recursion of
FindPageIndex().

A check for 0 |skip_count| is required before decrementing. |skip_count|
is unsigned so whenever this is decremented, the check against the
"Count" key of the "Pages" dictionary always succeeds and |skip_count|
is incorrectly reduced again.

Bug: pdfium:1506

Change-Id: If82b77ab6e1ba2ad44b8d138a7ebffecb079313a
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/68390
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>