Rsyslog: parameter "tmp_cursor" need free after used.

[ChenQianCQ]

https://github.com/rsyslog/rsyslog/blob/792f1e50e0bc2eac17cf7ee0b9a1a71c4a6b5707/plugins/imjournal/imjournal.c#L624

[rgerhards]

@ChenQianCQ thx for the bug report!

@jvymazal could you have a look or should I check?

[jvymazal]

looks correct (that is, the bug is there), will follow with PR.

[rgerhards]

@jvymazal thx a ton! Do you think we could try to reproduce this with a CI test (and valgrind?). That would be a nice addition.

[ChenQianCQ]

Maybe we need check all caller of "sd_journal_get_cursor". In function "persistJournalState" , it just get and replace global parameter "last_cursor", also not freed.

[rgerhards]

@ChenQianCQ thx again! Are you willing to submit a PR? That would really be phantastic! Also everyone would see from the commit history that you did great work. I am happy to help with get it going.

[lock[bot]]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.