Skip to content

Crash if the same wildcard source is used in multiple sources#5091

Merged
HofiOne merged 6 commits intosyslog-ng:masterfrom
HofiOne:crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090
Sep 12, 2024
Merged

Crash if the same wildcard source is used in multiple sources#5091
HofiOne merged 6 commits intosyslog-ng:masterfrom
HofiOne:crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090

Conversation

@HofiOne
Copy link
Collaborator

@HofiOne HofiOne commented Sep 10, 2024

Fixes: #5090

Signed-off-by: Hofi [email protected]

@HofiOne HofiOne force-pushed the crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090 branch 2 times, most recently from f58d82e to ab0b07c Compare September 10, 2024 16:03
@kira-syslogng
Copy link
Contributor

Build FAILURE

@HofiOne HofiOne force-pushed the crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090 branch from 8b9fe20 to e6fb2aa Compare September 11, 2024 11:43
@HofiOne HofiOne requested a review from kovgeri01 September 11, 2024 11:45
@syslog-ng syslog-ng deleted a comment from kira-syslogng Sep 11, 2024
@syslog-ng syslog-ng deleted a comment from kira-syslogng Sep 11, 2024
@syslog-ng syslog-ng deleted a comment from kira-syslogng Sep 11, 2024
@HofiOne HofiOne force-pushed the crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090 branch from e6fb2aa to dcbde6d Compare September 11, 2024 11:55
MOTE:
- removed the filepath from the persist names
- using a similar format, with a smaller memory footprint, both if the `persist-name()` option is set and if it is not

Signed-off-by: Hofi <[email protected]>
…for all the affile-sources automatically

Signed-off-by: Hofi <[email protected]>
@HofiOne HofiOne force-pushed the crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090 branch from dcbde6d to a96f02e Compare September 11, 2024 11:55
@HofiOne HofiOne marked this pull request as ready for review September 11, 2024 12:35
@kira-syslogng
Copy link
Contributor

Build FAILURE

@HofiOne
Copy link
Collaborator Author

HofiOne commented Sep 11, 2024

@kira-syslogng test this please;

@HofiOne HofiOne requested a review from kovgeri01 September 12, 2024 10:40
@HofiOne HofiOne merged commit e703ed7 into syslog-ng:master Sep 12, 2024
@HofiOne HofiOne deleted the crash-if-the-same-wildcard-source-is-used-in-multiple-sources-5090 branch September 12, 2024 12:31
HofiOne added a commit to HofiOne/syslog-ng that referenced this pull request Sep 18, 2024
HofiOne added a commit to HofiOne/syslog-ng that referenced this pull request Sep 18, 2024
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.

crash if the same wildcard-source is used in multiple sources

3 participants