OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-2-g0a9a6b5

Gerrit Code Review gerrit@openafs.org
Sat, 2 Sep 2017 15:41:41 -0400


The following commit has been merged in the master branch:
commit 0a9a6b57ce6e1c97fcc651c8cb74e66fc8422a1e
Author: Anders Kaseorg <andersk@mit.edu>
Date:   Fri Sep 1 23:37:07 2017 -0400

    vol: Fix two buffers being one char too short
    
    Fixes these warnings:
    
    namei_ops.c: In function 'namei_copy_on_write':
    namei_ops.c:1328:31: warning: 'snprintf' output may be truncated before the last format character [-Wformat-truncation=]
      snprintf(path, sizeof(path), "%s-tmp", name.n_path);
                                   ^~~~~~~~
    namei_ops.c:1328:2: note: 'snprintf' output between 5 and 260 bytes into a destination of size 259
      snprintf(path, sizeof(path), "%s-tmp", name.n_path);
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    vol_split.c: In function 'split_volume':
    vol_split.c:576:22: warning: 'sprintf' may write a terminating nul past the end of the destination [-Wformat-overflow=]
         sprintf(symlink, "#%s", V_name(newvol));
                          ^~~~~
    vol_split.c:576:5: note: 'sprintf' output between 2 and 33 bytes into a destination of size 32
         sprintf(symlink, "#%s", V_name(newvol));
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    Change-Id: If212ebc29fa3fe10fe1e2f70dfb5f7509c269ae9
    Reviewed-on: https://gerrit.openafs.org/12722
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>

 src/vol/namei_ops.c    |    2 +-
 src/volser/vol_split.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository