mirrors/netsurf
1
0
Fork 0
mirror of https://github.com/netsurf-browser/netsurf synced 2024-12-21 03:32:35 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix segmentation fault when a string option is set to NULL

Browse Source 
The check for a user option being a default value was not protecting
against the value being set to NULL.
This commit is contained in:
Vincent Sanders 2015-11-05 14:47:45 +00:00
parent b54174e54a
commit c3b7e650c5
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
utils/nsoption.c
View File

@ -16,7 +16,8 @@
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*/
/** \file
/**
* \file
* Option reading and saving (implementation).
*
* Options are stored in the format key:value, one per line.
@ -232,6 +233,8 @@ nsoption_is_set(const struct nsoption_s *opts,
*/
if (((defs[entry].value.s == NULL) &&
(opts[entry].value.s != NULL)) ||
((defs[entry].value.s != NULL) &&
(opts[entry].value.s == NULL)) ||
((defs[entry].value.s != opts[entry].value.s) &&
(strcmp(opts[entry].value.s, defs[entry].value.s) != 0))) {
ret = true;