diff options
author | Guillem Jover <guillem@debian.org> | 2009-02-27 04:55:23 +0200 |
---|---|---|
committer | Guillem Jover <guillem@debian.org> | 2009-02-27 05:31:26 +0200 |
commit | a2de4aa132890b97dbe3ec028322562bc2e5b033 (patch) | |
tree | 9c267ba7cd96f8a8b9f84a53a4ffc62f03c6a0f3 /lib | |
parent | 56dee78ddab32996e2b145b53c6ed46582860f10 (diff) | |
download | dpkg-a2de4aa132890b97dbe3ec028322562bc2e5b033.tar.gz |
libdpkg: Do nothing when parsing a boolean field with an empty string
The field should have been already initialized to 0 by
blankpackageperfile, so no need to assign again.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/fields.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/fields.c b/lib/fields.c index 31c9ae365..ce2b04846 100644 --- a/lib/fields.c +++ b/lib/fields.c @@ -130,10 +130,10 @@ void f_boolean(struct pkginfo *pigp, struct pkginfoperfile *pifp, enum parsedbflags flags, const char *filename, int lno, FILE *warnto, int *warncount, const char *value, const struct fieldinfo *fip) { - pifp->essential = *value ? - convert_string(filename, lno, _("yes/no in boolean field"), - -1, pigp, value, booleaninfos, NULL) : - 0; + if (!*value) + return; + pifp->essential = convert_string(filename, lno, _("yes/no in boolean field"), + -1, pigp, value, booleaninfos, NULL); } void f_section(struct pkginfo *pigp, struct pkginfoperfile *pifp, |