Suhrud Patankar has uploaded this change for review.

View Change

heap use after free for component_log_level

In function log_conf_commit:
2568    if (component_log_level != default_log_levels)
2569		gsh_free(component_log_level);
2570    component_log_level = logger->comp_log_level;

component_log_level is a global pointer.
While the current thread is freeing and re-setting, If any other
thread is accessing the pointer after execution of line 2569
but before 2570 is fully executed it will hit use-after-free case.
Avoid changing component_log_level once set.

Change-Id: If44236e323f39473c590930f4ce8bd4fd127c5f7
Signed-off-by: ashish.sangwan <ashish.sangwan@nutanix.com>
---
M src/log/log_functions.c
1 file changed, 12 insertions(+), 5 deletions(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/29/468729/1

To view, visit change 468729. To unsubscribe, or for help writing mail filters, visit settings.