aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@openbsd.org>2016-08-17 18:59:37 +0000
committerIngo Schwarze <schwarze@openbsd.org>2016-08-17 18:59:37 +0000
commit58f7533c9f4a757e9db71039330750a0b958f008 (patch)
treebec95ab4bba822a6de5ec68dd3a64ff6dbe6e52c
parent6fa9a1baff943f4d115847a9362593ac8855b30f (diff)
downloadmandoc-58f7533c9f4a757e9db71039330750a0b958f008.tar.gz
mandoc-58f7533c9f4a757e9db71039330750a0b958f008.tar.zst
mandoc-58f7533c9f4a757e9db71039330750a0b958f008.zip
When reading back a mandoc.db(5) file in order to apply incremental
changes, do not prepend a stray NAME_FILE (0x10) byte to the first names of pages. Bug found while investigating another issue reported by sthen@.
-rw-r--r--dba.c5
-rw-r--r--dba_read.c7
2 files changed, 6 insertions, 6 deletions
diff --git a/dba.c b/dba.c
index 5d190bca..b414c436 100644
--- a/dba.c
+++ b/dba.c
@@ -1,4 +1,4 @@
-/* $Id: dba.c,v 1.6 2016/08/17 18:10:39 schwarze Exp $ */
+/* $Id: dba.c,v 1.7 2016/08/17 18:59:37 schwarze Exp $ */
/*
* Copyright (c) 2016 Ingo Schwarze <schwarze@openbsd.org>
*
@@ -143,7 +143,8 @@ dba_page_new(struct dba_array *pages, const char *name, const char *sect,
page = dba_array_new(DBP_MAX, 0);
entry = dba_array_new(1, DBA_STR | DBA_GROW);
- dba_array_add(entry, prepend(name, NAME_FILE & NAME_MASK));
+ if (name != NULL)
+ dba_array_add(entry, prepend(name, NAME_FILE & NAME_MASK));
dba_array_add(page, entry);
entry = dba_array_new(1, DBA_STR | DBA_GROW);
dba_array_add(entry, (void *)sect);
diff --git a/dba_read.c b/dba_read.c
index bf003fa9..b24bd2e7 100644
--- a/dba_read.c
+++ b/dba_read.c
@@ -1,4 +1,4 @@
-/* $Id: dba_read.c,v 1.2 2016/08/04 09:33:57 schwarze Exp $ */
+/* $Id: dba_read.c,v 1.3 2016/08/17 18:59:37 schwarze Exp $ */
/*
* Copyright (c) 2016 Ingo Schwarze <schwarze@openbsd.org>
*
@@ -48,10 +48,9 @@ dba_read(const char *fname)
dba = dba_new(npages < 128 ? 128 : npages);
for (ip = 0; ip < npages; ip++) {
pdata = dbm_page_get(ip);
- page = dba_page_new(dba->pages, pdata->name, pdata->sect,
+ page = dba_page_new(dba->pages, NULL, pdata->sect,
pdata->arch, pdata->desc, pdata->file + 1, *pdata->file);
- cp = pdata->name;
- while (*(cp = strchr(cp, '\0') + 1) != '\0')
+ for (cp = pdata->name; *cp != '\0'; cp = strchr(cp, '\0') + 1)
dba_page_add(page, DBP_NAME, cp);
cp = pdata->sect;
while (*(cp = strchr(cp, '\0') + 1) != '\0')