Closed Bug 1406483 Opened 7 years ago Closed 6 years ago

firefox-56.0.1/modules/pdfium/pdfium/core/fpdfapi/edit/fpdf_edit_create.cpp:1424: bas assert ?

Categories

(Firefox :: PDF Viewer, defect)

Product:
Firefox
Component:
PDF Viewer
Version:
54 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: dcb314, Unassigned)

Details

User Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0 Build ID: 20170613080647 Steps to reproduce: firefox-56.0.1/modules/pdfium/pdfium/core/fpdfapi/edit/fpdf_edit_create.cpp:1424]: (warning) Logical disjunction always evaluates to true: m_iStage > -1 || m_iStage < 20. Source code is ASSERT(m_iStage > -1 || m_iStage < 20); maybe better code ASSERT(m_iStage > -1 && m_iStage < 20);
Component: Untriaged → PDF Viewer
Some duplicates: [firefox-56.0.1/modules/pdfium/pdfium/core/fpdfapi/edit/fpdf_edit_create.cpp:1521]: (warning) Logical disjunction always evaluates to true: m_iStage >= 20 || m_iStage < 30. [firefox-56.0.1/modules/pdfium/pdfium/core/fpdfapi/edit/fpdf_edit_create.cpp:1567]: (warning) Logical disjunction always evaluates to true: m_iStage >= 80 || m_iStage < 90.
Status: UNCONFIRMED → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.