diff options
| author | Karel Zak <kzak@redhat.com> | 2009-01-24 00:37:22 +0100 |
|---|---|---|
| committer | Kay Sievers <kay.sievers@vrfy.org> | 2009-01-24 15:20:37 +0100 |
| commit | 3e2084614b4d8308d5c0ceb34ec3634eb3353c00 (patch) | |
| tree | 9e8a9533371c2ed890d9125685dc5c38f7e7de4d | |
| parent | 9ad0d7370cffff0bc07e9c561559b4066ae39dc6 (diff) | |
vol_id: fix ddf version string
The version string in DDF suberblock does not include the null terminator
and the snprintf() function writes at most "size" bytes *including* the
null byte.
old version:
ID_FS_VERSION=02.00.0
new version:
ID_FS_VERSION=02.00.00
Signed-off-by: Karel Zak <kzak@redhat.com>
| -rw-r--r-- | extras/volume_id/lib/ddf_raid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extras/volume_id/lib/ddf_raid.c b/extras/volume_id/lib/ddf_raid.c index 58bdae94a7..32f20ced49 100644 --- a/extras/volume_id/lib/ddf_raid.c +++ b/extras/volume_id/lib/ddf_raid.c @@ -87,7 +87,7 @@ int volume_id_probe_ddf_raid(struct volume_id *id, uint64_t off, uint64_t size) return -1; found: volume_id_set_uuid(id, ddf->guid, DDF_GUID_LENGTH, UUID_STRING); - snprintf(id->type_version, DDF_REV_LENGTH, "%s", ddf->ddf_rev); + snprintf(id->type_version, DDF_REV_LENGTH + 1, "%s", ddf->ddf_rev); volume_id_set_usage(id, VOLUME_ID_RAID); id->type = "ddf_raid_member"; return 0; |