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.
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
common/address_help.cc: fix the leak in entity_addr_from_url() #6987
Conversation
LGTM but keep commit message title within 80 chars will be better |
@qiankunzheng could you update the commit message to add an before the tracker ticket number, like:
?
would be better. |
@tchaikov |
lgtm, would be ideal if jenkins can also nod on this. i grep'ed the repo, |
Fixes: #14132 Signed-off-by: Qiankun Zheng <zheng.qiankun@h3c.com>
@tchaikov |
@qiankunzheng sorry for the latency, the jenkins is disabled at this moment, merging it! |
common/address_help.cc: fix the leak in entity_addr_from_url() Reviewed-by: Kefu Chai <kchai@redhat.com>
Fixes:#14132
Signed-off-by: Qiankun Zheng zheng.qiankun@h3c.com