diff options
author | Nalin Dahyabhai <nalin@dahyabhai.net> | 2012-11-14 10:27:00 -0500 |
---|---|---|
committer | Nalin Dahyabhai <nalin@dahyabhai.net> | 2012-11-14 10:27:00 -0500 |
commit | d80ce0def40c2b4d1f8627791707ee074040920a (patch) | |
tree | ae33031c13f08db8452639891c1684d7bb1e73db /src | |
parent | d1d78630e01dfc64319ce4db406477379b1a2b37 (diff) | |
download | slapi-nis-d80ce0def40c2b4d1f8627791707ee074040920a.tar.gz slapi-nis-d80ce0def40c2b4d1f8627791707ee074040920a.tar.xz slapi-nis-d80ce0def40c2b4d1f8627791707ee074040920a.zip |
explicitly check for "no" values
Diffstat (limited to 'src')
-rw-r--r-- | src/back-shr.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/back-shr.c b/src/back-shr.c index a6a7230..97fa2e2 100644 --- a/src/back-shr.c +++ b/src/back-shr.c @@ -550,11 +550,19 @@ backend_shr_get_vattr_boolean(struct plugin_state *state, if (tmp != NULL) { /* FIXME: should we use nl_langinfo(YESEXPR) here? */ if ((strcasecmp(tmp, "yes") == 0) || + (strcasecmp(tmp, "true") == 0) || (strcasecmp(tmp, "on") == 0) || (strcasecmp(tmp, "1") == 0)) { ret = TRUE; } else { - ret = FALSE; + if ((strcasecmp(tmp, "no") == 0) || + (strcasecmp(tmp, "false") == 0) || + (strcasecmp(tmp, "off") == 0) || + (strcasecmp(tmp, "0") == 0)) { + ret = FALSE; + } else { + ret = default_value; + } } free(tmp); } |