Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[podofo-0.10.0]a SIGSEGV error occurs #67

Closed
longuu9 opened this issue Apr 11, 2023 · 4 comments
Closed

[podofo-0.10.0]a SIGSEGV error occurs #67

longuu9 opened this issue Apr 11, 2023 · 4 comments

Comments

@longuu9
Copy link

longuu9 commented Apr 11, 2023

When using podofopdfinfo to parse a PDF file, a SIGSEGV error occurs. By debugging with gdb, it was found that the error occurred at line 163 in podofo-0.10.0/src/podofo/main/PdfObject.cpp:

if (m_IsDelayedLoadDone)

When checking the value of m_IsDelayedLoadDone with "p" command, it was found that the value was 0x31. As a boolean value, it should only be assigned either 0 or 1, but not any other numbers. Previously, PoDoFo::PdfObject::DelayedLoad was also called and executed normally, but calling this function in the getString() function would result in a failure. The specific gdb bt stack trace is as follows.

Z_%VS`9LQ_22SJVY06P7ZFB

Command Input

podofopdfinfo poc_file

poc_file are attached.

Environment

  • OS: Ubuntu 20.04.1
  • clang:12.0.0
  • podofo:0.10.0

poc_file.zip

@ceztko ceztko added tools The issue is tool related Delayed Merging for this PR is delayed until a better time opportunity arises labels Apr 11, 2023
@ceztko
Copy link
Contributor

ceztko commented Apr 11, 2023

@longuu9
Copy link
Author

longuu9 commented Apr 11, 2023

Although we discovered this bug while using podofopdfinfo, the issue is actually occurring in the core code of the PoDoFo library, rather than in the PoDoFo tools.

@ceztko ceztko removed tools The issue is tool related Delayed Merging for this PR is delayed until a better time opportunity arises labels Apr 11, 2023
@ceztko
Copy link
Contributor

ceztko commented Apr 11, 2023

Sorry, that was too quick triage, but I am actually not expecting people to use tools at all in this moment so I am a bit surprised to see them mentioned in bug reports.

@ceztko
Copy link
Contributor

ceztko commented Apr 12, 2023

Should be fixed with 3759eb6 .

@ceztko ceztko closed this as completed Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants