Do not use vsnprintf twice

Change-Id: Icb20f77ee2c2992b0fe91b814705f2eba7e4ad8d
Signed-off-by: cheoleun moon <chleun.moon@samsung.com>

diff --git a/packaging/capi-network-vine.spec b/packaging/capi-network-vine.spec
index ef31f5b..8099c81 100755 (executable)
--- a/packaging/capi-network-vine.spec
+++ b/packaging/capi-network-vine.spec
@@ -3,7 +3,7 @@
 %bcond_without use_glib_event_loop
 Name:    capi-network-vine
 Summary: An service discovery framework
-Version: 1.1.10
+Version: 1.2.0
 Release: 0
 Group:   Network & Connectivity/API
 License: Apache-2.0

diff --git a/src/logger/vine-log.c b/src/logger/vine-log.c
index 8f5d1f1..1cefebb 100755 (executable)
--- a/src/logger/vine-log.c
+++ b/src/logger/vine-log.c
@@ -70,11 +70,9 @@ API void vine_set_logger(vine_logger func)
 }
 
 #if defined(TIZEN_OS) && defined(USE_DLOG)
-static void __write_tizen_dlog(int log_level, const char *format, va_list args)
+static void __write_tizen_dlog(int log_level, const char *buf)
 {
        int dlog_level = 0;
-       char buf[VINE_MAX_LOG_LEN + 1] = {0, };
-       vsnprintf(buf, VINE_MAX_LOG_LEN, format, args);
        switch (log_level) {
        case VINE_LOG_DEBUG:
                dlog_level = DLOG_DEBUG;
@@ -90,13 +88,11 @@ static void __write_tizen_dlog(int log_level, const char *format, va_list args)
 }
 #endif // TIZEN_OS && USE_DLOG
 
-static void __write_vine_logger(int log_level, const char *format, va_list args)
+static void __write_vine_logger(int log_level, const char *buf)
 {
        if (!__log_func)
                return;
 
-       char buf[VINE_MAX_LOG_LEN + 1] = {0, };
-       vsnprintf(buf, VINE_MAX_LOG_LEN, format, args);
        __log_func(log_level, buf);
 }
 
@@ -108,10 +104,13 @@ void _vine_log(int log_level, const char *format, ...)
        va_list args;
        va_start(args, format);
 
+       char buf[VINE_MAX_LOG_LEN + 1] = {0, };
+       vsnprintf(buf, VINE_MAX_LOG_LEN, format, args);
+
 #if defined(TIZEN_OS) && defined(USE_DLOG)
-       __write_tizen_dlog(log_level, format, args);
+       __write_tizen_dlog(log_level, buf);
 #endif
-       __write_vine_logger(log_level, format, args);
+       __write_vine_logger(log_level, buf);
 
        va_end(args);
 }