ulog组件中ulog_voutput(...)函数的bug反馈

发布于 2021-01-12 09:19:58 浏览：1834 订阅该版

void ulog_voutput(rt_uint32_t level, const char *tag, rt_bool_t newline, const char *format, va_list args)
{
    char *log_buf = NULL;
    rt_size_t log_len = 0;

... ...

RT_ASSERT(tag);
    RT_ASSERT(format);

if (!ulog.init_ok)
    {
        return;
    }

... ...

/* get log buffer 获取存储log使用buf地址 */
    log_buf = get_log_buf();

/* lock output 上锁 */
    output_lock();

/* 格式化处理 */
    log_len = ulog_formater(log_buf, level, tag, newline, format, args);

... ...

/* do log output 输出log */
    do_output(level, tag, RT_FALSE, log_buf, log_len);

/* 这里是我自己添加的代码；
	 * 如果某个线程第一次输出的log较长，不在这里清零log_buf的话；有其它线程使用API进行log输出时，会发生问题；
	 * 例如: LOG_I("Main Test Integer %d", cnt);
	 *       LOG_I("ulog test application, hello world");
	 * 打印结果如下：
	 *          I/MAIN: Main Test Integer 1
		        I/MAIN: ulog test application, hello world
		        I/MAIN: Main Test Integer 2
		        , hello world
		        I/MAIN: ulog test application, hello world
		        I/MAIN: Main Test Integer 3
		        , hello world
		        I/MAIN: ulog test application, hello world
	 * 问题原因就出在"ulog_formater()"函数中，由于上次的log较长，也没有清理log_buf，vsnprintf函数返回计算长度的时候发生问题；
	*/
    rt_memset((void *)log_buf, '\0', log_len);

/* unlock output */
    output_unlock();
}
追加：
当开启RT_USING_DEVICE宏进行测试，源码是没有问题的；不开启此宏测试有问题。