>From Kinglong Mee <kinglong...@gmail.com>:

Kinglong Mee has uploaded this change for review. ( 
https://review.gerrithub.io/386776


Change subject: config_parsing: reset err_type->dispose at err_out in proc_block
......................................................................

config_parsing: reset err_type->dispose at err_out in proc_block

If don't reset err_type->dispose to false when committing fail,
it will pollute the next entry.

For example, if the committing of FSAL failed, a double free may happens.
EXPORT {
 FSAL {
  Name = GLUSTER;
  ... ...
 }
 CLIENT {
  ... ...
 }
}

0  0x00007f056cebb50a in FreeClientList ()
1  0x00007f056cebe035 in free_export_resources ()
2  0x00007f056cecdc53 in free_export ()
3  0x00007f056cebb5f4 in export_init ()
4  0x00007f056cef6f0d in proc_block ()
5  0x00007f056cef7ab0 in load_config_from_parse ()
6  0x00007f056cebd9db in ReadExports ()
7  0x00007f056ce0b5e1 in main ()

Change-Id: Ief54601a3d1b7559b7f132b720095b811f2eddf2
Signed-off-by: Kinglong Mee <mijinl...@open-fs.com>
---
M src/config_parsing/config_parsing.c
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/76/386776/1
-- 
To view, visit https://review.gerrithub.io/386776
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ief54601a3d1b7559b7f132b720095b811f2eddf2
Gerrit-Change-Number: 386776
Gerrit-PatchSet: 1
Gerrit-Owner: Kinglong Mee <kinglong...@gmail.com>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to