In ss_name(), we should check return value of malloc(), otherwise, it may cause a segmentation fault problem. Signed-off-by: Zhiqiang Liu <liuzhiqiang26@xxxxxxxxxx> Signed-off-by: Wu Guanghao <wuguanghao3@xxxxxxxxxx> --- lib/ss/error.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/ss/error.c b/lib/ss/error.c index 8d345a9f..656b71be 100644 --- a/lib/ss/error.c +++ b/lib/ss/error.c @@ -42,6 +42,8 @@ char *ss_name(int sci_idx) (strlen(infop->subsystem_name)+ strlen(infop->current_request)+ 4)); + if (ret_val == (char *)NULL) + return ((char *)NULL); cp = ret_val; cp1 = infop->subsystem_name; while (*cp1) --
