diff options
| author | Jim Meyering | 2012-10-04 13:09:48 +0200 |
|---|---|---|
| committer | Anthony Liguori | 2012-10-05 14:58:36 +0200 |
| commit | d66f8e7bd3de4a2ecf0680c635f870f2138425b8 (patch) | |
| tree | cbca695676fd55f235d66ff6fd329fc4330631f5 /block | |
| parent | sheepdog: avoid a few buffer overruns (diff) | |
| download | qemu-d66f8e7bd3de4a2ecf0680c635f870f2138425b8.tar.gz qemu-d66f8e7bd3de4a2ecf0680c635f870f2138425b8.tar.xz qemu-d66f8e7bd3de4a2ecf0680c635f870f2138425b8.zip | |
vmdk: relative_path: use pstrcpy in place of strncpy
Avoid strncpy+manual-NUL-terminate. Use pstrcpy instead.
Acked-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'block')
| -rw-r--r-- | block/vmdk.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/block/vmdk.c b/block/vmdk.c index f2e861b074..1a80e5a247 100644 --- a/block/vmdk.c +++ b/block/vmdk.c @@ -1408,8 +1408,7 @@ static int relative_path(char *dest, int dest_size, return -1; } if (path_is_absolute(target)) { - dest[dest_size - 1] = '\0'; - strncpy(dest, target, dest_size - 1); + pstrcpy(dest, dest_size, target); return 0; } while (base[i] == target[i]) { |