Skip to content
/ server Public

Mroonga: fix SIGSEGV on NULL mroonga_log_file#4801

Open
hadeer-r wants to merge 1 commit intoMariaDB:10.6from
hadeer-r:MDEV-37952-log_file
Open

Mroonga: fix SIGSEGV on NULL mroonga_log_file#4801
hadeer-r wants to merge 1 commit intoMariaDB:10.6from
hadeer-r:MDEV-37952-log_file

Conversation

@hadeer-r
Copy link

@hadeer-r hadeer-r commented Mar 13, 2026

  • Fix SIGSEGV when mroonga_log_file is set to NULL or empty string
  • Add mroonga_log_file_check() to reject NULL/empty values
  • Add MTR test to cover this case

This fix is related to MDEV-37952

@hadeer-r hadeer-r marked this pull request as ready for review March 13, 2026 17:25
Add mrn_log_file_check() to validate that
mroonga_log_file is not set to NULL or an empty
string, which previously caused a segfault.
@hadeer-r hadeer-r force-pushed the MDEV-37952-log_file branch from ebad0e0 to f7ce93e Compare March 14, 2026 10:05
kou added a commit to mroonga/mroonga that referenced this pull request Mar 16, 2026
- Fix SIGSEGV when mroonga_log_file is set to NULL or empty string
- Add mroonga_log_file_check() to reject NULL/empty values
- Add MTR test to cover this case

this same fix I submitted to MariaDB:
MariaDB/server#4801

---------

Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
Copy link
Contributor

@kou kou left a comment

Choose a reason for hiding this comment

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

+1

@gkodinov gkodinov added the External Contribution All PRs from entities outside of MariaDB Foundation, Corporation, Codership agreements. label Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

External Contribution All PRs from entities outside of MariaDB Foundation, Corporation, Codership agreements.

Development

Successfully merging this pull request may close these issues.

3 participants