summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorGuillem Jover <guillem@debian.org>2009-02-27 04:55:23 +0200
committerGuillem Jover <guillem@debian.org>2009-02-27 05:31:26 +0200
commita2de4aa132890b97dbe3ec028322562bc2e5b033 (patch)
tree9c267ba7cd96f8a8b9f84a53a4ffc62f03c6a0f3 /lib
parent56dee78ddab32996e2b145b53c6ed46582860f10 (diff)
downloaddpkg-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.c8
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,