Potential null dereference in ngx_time.c · nginx/nginx

(3 条评论) (0 个反应) (1 位负责人)C (7,903 个派生)batch import

Complexity - hardbuggood first issue

仓库指标

星标: (30,331 个星标)
PR 合并指标: (平均合并 27天 8小时) (30 天内合并 17 个 PR)

描述

In results of static analyze of nginx sources I founded potential null dereference

https://github.com/nginx/nginx/blob/00637cce366f17b78fe1ed5c1ef0e534143045f6/src/os/unix/ngx_time.c#L48-L50

localtime() function potentially can return NULL value, which will dereference in strftime(). How about to add check for this case?

贡献者指南

研究方向: 查看提供的链接中的代码（ngx time.c 第 48-50 行）。在将 localtime() 的返回值传递给 strftime() 之前，实现对其返回值的空值检查。如果考虑线程安全，可以考虑使用 localtime r()。确保修复不会引入任何回归问题。
技术栈: c
领域: backend
议题类型: 缺陷
难度: 2
预计时间: 1 小时以内
活动状态: 已阻塞
清晰度: 清晰
前置要求: C programmingunderstanding of localtime()
新手友好度: 70