Skip to content

[libpas] pas_report_crash_extract_pgm_failure should use the supplied task id when invoking the reader callback#61835

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
dhecht:eng/libpas-pas_report_crash_extract_pgm_failure-should-use-the-supplied-task-id-when-invoking-the-reader-callback
Apr 2, 2026
Merged

[libpas] pas_report_crash_extract_pgm_failure should use the supplied task id when invoking the reader callback#61835
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
dhecht:eng/libpas-pas_report_crash_extract_pgm_failure-should-use-the-supplied-task-id-when-invoking-the-reader-callback

Conversation

@dhecht
Copy link
Copy Markdown
Contributor

@dhecht dhecht commented Apr 1, 2026

f7b2175

[libpas] pas_report_crash_extract_pgm_failure should use the supplied task id when invoking the reader callback
https://bugs.webkit.org/show_bug.cgi?id=311270
rdar://173765415

Reviewed by Marcus Plutowski.

ReportCrash supplies a task id to use when calling back. The libpas
code should use that when invoking the callbacks.

Added test to PGMTests.cpp to verify the supplied task is used.

* Source/bmalloc/libpas/libpas.xcodeproj/project.pbxproj:
* Source/bmalloc/libpas/src/libpas/pas_report_crash.c:
(pas_enumerator_reader_adapter):
(pas_report_crash_extract_pgm_failure):
(setup_enumerator_for_crash_reporting): Deleted.
* Source/bmalloc/libpas/src/test/PGMTests.cpp:
(std::mockCrashReporterReader):
(std::testPGMCrashReportTaskPassthrough):
(addPGMTests):

Canonical link: https://commits.webkit.org/310473@main

8166c5c

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows Apple Internal
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe 💥 🛠 win ⏳ 🛠 ios-apple
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 💥 🧪 win-tests ⏳ 🛠 mac-apple
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe ⏳ 🛠 vision-apple
✅ 🧪 ios-wk2-wpt ✅ 🧪 api-mac-debug ✅ 🛠 gtk3-libwebrtc
✅ 🛠 🧪 jsc ✅ 🧪 api-ios ✅ 🧪 mac-wk1 ✅ 🛠 gtk
✅ 🛠 🧪 jsc-debug-arm64 ✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🛠 playstation
✅ 🛠 🧪 unsafe-merge ✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2 ✅ 🛠 jsc-armv7
✅ 🛠 tv ✅ 🧪 jsc-armv7-tests
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@dhecht dhecht requested a review from a team as a code owner April 1, 2026 16:38
@dhecht dhecht self-assigned this Apr 1, 2026
@dhecht dhecht added the bmalloc For bugs in bmalloc label Apr 1, 2026
@stwrt stwrt requested review from stwrt and zakariaridouh April 1, 2026 16:47
Copy link
Copy Markdown
Contributor

@Achierius Achierius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me

@zakariaridouh
Copy link
Copy Markdown
Contributor

This looks great! Thanks for adding a test as well.

@dhecht dhecht added safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing and removed safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks labels Apr 2, 2026
… task id when invoking the reader callback

https://bugs.webkit.org/show_bug.cgi?id=311270
rdar://173765415

Reviewed by Marcus Plutowski.

ReportCrash supplies a task id to use when calling back. The libpas
code should use that when invoking the callbacks.

Added test to PGMTests.cpp to verify the supplied task is used.

* Source/bmalloc/libpas/libpas.xcodeproj/project.pbxproj:
* Source/bmalloc/libpas/src/libpas/pas_report_crash.c:
(pas_enumerator_reader_adapter):
(pas_report_crash_extract_pgm_failure):
(setup_enumerator_for_crash_reporting): Deleted.
* Source/bmalloc/libpas/src/test/PGMTests.cpp:
(std::mockCrashReporterReader):
(std::testPGMCrashReportTaskPassthrough):
(addPGMTests):

Canonical link: https://commits.webkit.org/310473@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/libpas-pas_report_crash_extract_pgm_failure-should-use-the-supplied-task-id-when-invoking-the-reader-callback branch from 8166c5c to f7b2175 Compare April 2, 2026 23:06
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 310473@main (f7b2175): https://commits.webkit.org/310473@main

Reviewed commits have been landed. Closing PR #61835 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit f7b2175 into WebKit:main Apr 2, 2026
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Apr 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bmalloc For bugs in bmalloc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants