Summary: amfd: Correct coverity detected errors [#680]
Review request for Trac Ticket(s): #680
Peer Reviewer(s): AMF devs
Pull request to: AMF maints
Affected branch(es): default
Development branch: default
Impacted area Impact y/n
src/amf/amfd/ntf.cc | 4
1 files changed, 4 insertions(+), 0 deletions(-)
Problem:
Coverity check reported some RESOURCE_LEAK issues on amfd service.
- Variable job going out of scope leaks the storage it points to.
Fix:
Delete the variable before exiting the function.
diff --git
src/log/README|4 +-
src/log/README_LONGDN | 15 +---
src/log/logd/lgs_cb.h |2 -
src/log/logd/lgs_clm.cc | 25 +++-
src/log/logd/lgs_evt.cc | 123 ++---
src/log/logd/lgs_evt.h|7 ++
src/log/logd/lgs_mbcsv.cc
Summary:logd: replace client DB patricia tree with cpp Map [#1984]
Review request for Trac Ticket(s): #1984
Peer Reviewer(s):Vu, Lennart, Canh
Pull request to: <>
Affected branch(es): <>
Development branch: <>
Impacted area Impact y/n
src/imm/immnd/immnd_evt.c | 35 ---
src/imm/immnd/immnd_proc.c | 22 ++
2 files changed, 34 insertions(+), 23 deletions(-)
When SC comes back too fast, it will fail to change to SEVER_READY state
because immnd_proc_server() is not
Summary: amfd: Correct coverity detected errors [#680]
Review request for Trac Ticket(s): #680
Peer Reviewer(s): AMF devs
Pull request to: AMF maints
Affected branch(es): default
Development branch: default
Impacted area Impact y/n
src/amf/amfd/ntf.cc | 4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/src/amf/amfd/ntf.cc b/src/amf/amfd/ntf.cc
--- a/src/amf/amfd/ntf.cc
+++ b/src/amf/amfd/ntf.cc
@@ -630,6 +630,7 @@ uint32_t sendAlarmNotificationAvd(AVD_CL
if (status != SA_AIS_OK) {
Hi Zoran,
I will fix the problem and send new patch for review.
Thanks,
Hung Nguyen - DEK Technologies
From: Zoran Milinkovic zoran.milinko...@ericsson.com
Sent: Friday, February 10, 2017 8:49PM
To: Hung Nguyen,