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

fuzzer, clientsession: fix build #9026

Merged
merged 1 commit into from May 13, 2024
Merged

fuzzer, clientsession: fix build #9026

merged 1 commit into from May 13, 2024

Conversation

vmiklos
Copy link
Contributor

@vmiklos vmiklos commented May 10, 2024

Probably went wrong in commit de98583
(wsd: simplify DocumentBroker construction, 2024-04-03).

Also init logging to avoid:

common/Log.cpp:664:16: runtime error: downcast of address 0x6070000005d0 which does not point to an object of type 'GenericLogger'
0x6070000005d0: note: object is of type 'Poco::Logger'

similar to how COOLWSD::innerInitialize() does it.

Signed-off-by: Miklos Vajna [email protected]
Change-Id: Ic16bb2f070e1f0e12204bcc2fc70303d8c1250c8

@vmiklos
Copy link
Contributor Author

vmiklos commented May 10, 2024

Quoting https://cpci.cbg.collabora.co.uk:8080/job/github_online_master_debug_vs_co-24.04_cypress_desktop/737/console:

Test failed: integration_tests/desktop/impress/jsdialog_spec.js

retry.

@vmiklos
Copy link
Contributor Author

vmiklos commented May 10, 2024

Quoting https://cpci.cbg.collabora.co.uk:8080/job/github_online_master_debug_vs_co-24.04_cypress_mobile/689/console:

Test failed: integration_tests/mobile/impress/apply_font_shape_spec.js

retry.

@vmiklos
Copy link
Contributor Author

vmiklos commented May 13, 2024

Rebase to be on top of 6bc7e3a.

Probably went wrong in commit de98583
(wsd: simplify DocumentBroker construction, 2024-04-03).

Also init logging to avoid:

common/Log.cpp:664:16: runtime error: downcast of address 0x6070000005d0 which does not point to an object of type 'GenericLogger'
0x6070000005d0: note: object is of type 'Poco::Logger'

similar to how COOLWSD::innerInitialize() does it.

Signed-off-by: Miklos Vajna <[email protected]>
Change-Id: Ic16bb2f070e1f0e12204bcc2fc70303d8c1250c8
@vmiklos
Copy link
Contributor Author

vmiklos commented May 13, 2024

Quoting https://cpci.cbg.collabora.co.uk:8080/job/github_online_master_debug_vs_co-24.04/1419/console:

Test failed on unit-save-torture

retry.

@vmiklos vmiklos requested a review from caolanm May 13, 2024 07:14
@vmiklos
Copy link
Contributor Author

vmiklos commented May 13, 2024

@caolanm could you please review this? Thanks.

Fix build + make it not instantly crash locally when I try it.

@caolanm caolanm merged commit 0ef5e74 into master May 13, 2024
14 checks passed
@caolanm caolanm deleted the private/vmiklos/master branch May 13, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants