Skip to content

few leaks find via sanitizer #2696

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 16 commits into from
May 3, 2019
Merged

few leaks find via sanitizer #2696

merged 16 commits into from
May 3, 2019

Conversation

Kokan
Copy link
Collaborator

@Kokan Kokan commented Apr 26, 2019

No description provided.

Kokan added 16 commits April 26, 2019 22:44
@Kokan
cmake: extend ut fail rule for criterion
3929bca 
Signed-off-by: Kokan <[email protected]>
@Kokan
test: lexer free args, generator
41f37ff 
Signed-off-by: Kokan <[email protected]>
@Kokan
logmsg: test fix leaked message
d562c7e 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_tags: get_tag_by_id local
3c4159a 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_tags: expand *not* local variable
5f3af21 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_tags: leaked *name* variable
668bec8 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_framed_server: free proto
c500d86 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_graphite_output: split test
703405e 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_graphite_output: remove syslogformat dep
1b665c9 
Signed-off-by: Kokan <[email protected]>
@Kokan
graphite: fix leaked unixtime str
d064b85 
Signed-off-by: Kokan <[email protected]>
@Kokan
cfg: fix *yesno* value leak
81f64ce 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_csvparser_statistics: properly free parser
251ffc1 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_csvparser_statistics: remove syslogformat dep
af3a13c 
Signed-off-by: Kokan <[email protected]>
@Kokan
test_transport_factory_id: free ids
3117ff5 
Signed-off-by: Kokan <[email protected]>
@Kokan
string-list: small refactor use g_list_free_full
a4e14e4 
Signed-off-by: Kokan <[email protected]>
@Kokan
string-list: GPOINTER_TO_INT incorrect convertion
2ad3d14 
Signed-off-by: Kokan <[email protected]>
@kira-syslogng
Copy link
Contributor

Build SUCCESS

@alltilla alltilla self-requested a review May 2, 2019 08:42
alltilla
alltilla approved these changes May 3, 2019
View reviewed changes
Copy link
Collaborator

@alltilla alltilla left a comment

Choose a reason for hiding this comment

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

Really good fixes, thanks @Kokan ! :)

@alltilla alltilla merged commit 4617a56 into syslog-ng:master May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@furiel furiel furiel approved these changes

@alltilla alltilla alltilla approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Kokan @kira-syslogng @furiel @alltilla