From: Derrick Brashear <shadow@dementia.org>
Date: Wed, 21 Jul 2010 16:06:39 +0000 (-0400)
Subject: vlserver - Don't overflow buffer with backup pattern
X-Git-Tag: openafs-devel-1_5_76~77
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=d64948ac30d3aefda098727a2158406144877654;p=packages%2Fo%2Fopenafs.git

vlserver - Don't overflow buffer with backup pattern

based on a report from Hans-Werner Paulsen

avoid adding regex anchors to a buffer which is already full
when copying it to a like sized buffer. instead, leave extra space.

Change-Id: I486c51779da285888fe38ed2aa5f30dd2d008a9a
Reviewed-on: http://gerrit.openafs.org/2456
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/vlserver/vlprocs.c b/src/vlserver/vlprocs.c
index 0bd0c018a..4668faf68 100644
--- a/src/vlserver/vlprocs.c
+++ b/src/vlserver/vlprocs.c
@@ -1421,7 +1421,7 @@ SVL_ListAttributesN2(struct rx_call *rxcall,
     int pollcount = 0;
     int namematchRWBK, namematchRO, thismatch;
     int matchtype = 0;
-    char volumename[VL_MAXNAMELEN];
+    char volumename[VL_MAXNAMELEN+2]; /* regex anchors */
     char rxstr[AFS_RXINFO_LEN];
 #ifdef HAVE_POSIX_REGEX
     regex_t re;