From 5f918de77f94e993a3deeac97cc1029aa6ccd85c Mon Sep 17 00:00:00 2001 From: Stelian Pop Date: Mon, 24 Dec 2001 12:28:38 +0000 Subject: [PATCH] Fixed an off-by-one calculation when dumping a one letter subdirectory. --- CHANGES | 6 +++++- dump/optr.c | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/CHANGES b/CHANGES index 7ac3932..73ca931 100644 --- a/CHANGES +++ b/CHANGES @@ -1,4 +1,4 @@ -$Id: CHANGES,v 1.148 2001/11/30 10:14:36 stelian Exp $ +$Id: CHANGES,v 1.149 2001/12/24 12:28:38 stelian Exp $ Changes between versions 0.4b25 and 0.4b26 (released ?????????????????) ======================================================================= @@ -8,6 +8,10 @@ Changes between versions 0.4b25 and 0.4b26 (released ?????????????????) automatic (cron based) full and incremental dumping of several filesystems on a separate filesystem. +2. Fixed a off-by-one miscalculation which disabled dumping + a one letter subdirectory. Thanks to Chris Siebenmann + for reporting the bug. + Changes between versions 0.4b24 and 0.4b25 (released November 17, 2001) ======================================================================= diff --git a/dump/optr.c b/dump/optr.c index 56e1f34..6b86b9a 100644 --- a/dump/optr.c +++ b/dump/optr.c @@ -41,7 +41,7 @@ #ifndef lint static const char rcsid[] = - "$Id: optr.c,v 1.27 2001/09/12 09:02:51 stelian Exp $"; + "$Id: optr.c,v 1.28 2001/12/24 12:28:39 stelian Exp $"; #endif /* not lint */ #include @@ -539,7 +539,7 @@ fstabsearchdir(const char *key, char *directory) for (pf = table; pf != NULL; pf = pf->pf_next) { fs = pf->pf_fstab; if (strlen(fs->fs_file) > size && - strlen(key) > strlen(fs->fs_file) + 1 && + strlen(key) > strlen(fs->fs_file) && strncmp(fs->fs_file, key, strlen(fs->fs_file)) == 0 && (key[strlen(fs->fs_file)] == '/' || fs->fs_file[strlen(fs->fs_file) - 1] == '/')) { -- 2.39.2