Hi Neel, Reviewed the patch. Ack from me.
BR, Hung Nguyen - DEK Technologies -------------------------------------------------------------------------------- From: Neelakanta Reddy [email protected] Sent: Wednesday, December 30, 2015 8:34PM To: Zoran Milinkovic, Hung Nguyen [email protected], [email protected] Cc: Opensaf-devel [email protected] Subject: [PATCH 1 of 1] immtools: va_copy must have a correponding va_end for defaultImmutilError [#1654] osaf/tools/safimm/src/immutil.c | 1 + samples/immsv/immutils/immutil.c | 1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/osaf/tools/safimm/src/immutil.c b/osaf/tools/safimm/src/immutil.c --- a/osaf/tools/safimm/src/immutil.c +++ b/osaf/tools/safimm/src/immutil.c @@ -69,6 +69,7 @@ static void defaultImmutilError(char con va_copy(ap2, ap); vfprintf(stderr, fmt, ap); vsyslog(LOG_ERR, fmt, ap2); + va_end(ap2); va_end(ap); abort(); } diff --git a/samples/immsv/immutils/immutil.c b/samples/immsv/immutils/immutil.c --- a/samples/immsv/immutils/immutil.c +++ b/samples/immsv/immutils/immutil.c @@ -69,6 +69,7 @@ static void defaultImmutilError(char con va_copy(ap2, ap); vfprintf(stderr, fmt, ap); vsyslog(LOG_ERR, fmt, ap2); + va_end(ap2); va_end(ap); abort(); } ------------------------------------------------------------------------------ _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
