fix(consensus): free allocated CStrings

[thechampagne]

Goals:

• resolves #ISSUE-NUMBER

• ...

• ...

• ...

• ...

• check and update documentation. See guide and ask the team.

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +0.03% 🎉

Comparison is base (3652cfd) 77.01% compared to head (40585fc) 77.05%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6054      +/-   ##
==========================================
+ Coverage   77.01%   77.05%   +0.03%     
==========================================
  Files         145      145              
  Lines       13879    13879              
==========================================
+ Hits        10689    10694       +5     
+ Misses       3190     3185       -5     

Flag	Coverage Δ
jina	77.05% <0.00%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
...ina/serve/runtimes/gateway/graph/topology_graph.py	71.81% <0.00%> (+0.81%)	⬆️

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JoanFM]

Thank you for the contribution, may I ask how did you spot this error? there may be others in the code.

Thanks

[JoanFM]

Would this also require change?

LogLevel                 = C.CString(raftDefaultConfig.LogLevel)

[thechampagne]

Would this also require change?

Yes it should be freed, totally forgot it.

[thechampagne]

Thank you for the contribution, may I ask how did you spot this error? there may be others in the code.

Thanks

Reading the code 💀 not all of it for sure.