Skip to content

poppler: fix build #13530

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

poppler: fix build #13530

wants to merge 1 commit into from

Conversation

DavidKorczynski
Copy link
Collaborator

No description provided.

@DavidKorczynski
Copy link
Collaborator Author

@tsdgeos do you approve?

Signed-off-by: David Korczynski <david@adalogics.com>
@tsdgeos
Copy link
Contributor

tsdgeos commented Jul 6, 2025

Is this another occasion of #13289 happening?

I can't even see a build failure in https://issues.oss-fuzz.com/issues?q=poppler%20status:open

Where can i find the build failure log?

Aren't both functions declared in the same header?

tsdgeos@xps15:/usr/include/glib-2.0:$ wcgrep g_sort_array
./glib/gqsort.h:46:void g_sort_array (const void *array,

tsdgeos@xps15:/usr/include/glib-2.0:$ wcgrep g_qsort_with_data
./glib/gqsort.h:39:void g_qsort_with_data (gconstpointer pbase,

@DavidKorczynski now that i have your attention can we get #13435 merged please?

@DavidKorczynski
Copy link
Collaborator Author

Here are the build logs: https://oss-fuzz-build-logs.storage.googleapis.com/index.html#poppler

Am not entirely sure why it's not showing up on issues.oss-fuzz.com -- @oliverchang @jonathanmetzman

Looking at the logs, it looks like the most recent one is complaining about some ASAN issues, which I believe is different to the fix from here. I'll need to double check if this is only an issue in honggfuzz.

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

Successfully merging this pull request may close these issues.

2 participants