diff options
| author | Jim Meyering <meyering@redhat.com> | 2012-06-20 13:17:49 +0200 | 
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-07-02 15:43:46 -0700 | 
| commit | 237d32b784cfb0ff93f7f1081d89809af962ed56 (patch) | |
| tree | 52df6aca00b51a6be63cf97f7db33fbec617e2c1 /libglusterfsclient | |
| parent | ee3ed7c0bfcd340cd747a20812062925d5550d2a (diff) | |
libgf_init_vmpentry: assert strlen(vmp) > 0 to guard against using vmp[-1]
Change-Id: I091e6066fd470e9e0b67a00464b5713b86d3783c
BUG: 789278
Signed-off-by: Jim Meyering <meyering@redhat.com>
Reviewed-on: http://review.gluster.com/3599
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'libglusterfsclient')
| -rwxr-xr-x | libglusterfsclient/src/libglusterfsclient.c | 3 | 
1 files changed, 2 insertions, 1 deletions
diff --git a/libglusterfsclient/src/libglusterfsclient.c b/libglusterfsclient/src/libglusterfsclient.c index b0ae61dad31..e09dc39aa3b 100755 --- a/libglusterfsclient/src/libglusterfsclient.c +++ b/libglusterfsclient/src/libglusterfsclient.c @@ -1443,7 +1443,7 @@ struct vmp_entry *  libgf_init_vmpentry (char *vmp, glusterfs_handle_t *vmphandle)  {          struct vmp_entry        *entry = NULL; -        int                     vmplen = 0; +        size_t                  vmplen = 0;          int                     appendslash = 0;          int                     ret = -1; @@ -1454,6 +1454,7 @@ libgf_init_vmpentry (char *vmp, glusterfs_handle_t *vmphandle)          }          vmplen = strlen (vmp); +        assert (vmplen > 0);          if (vmp[vmplen - 1] != '/') {                  vmplen++;                  appendslash = 1;  | 
