diff options
author | John Snow <jsnow@redhat.com> | 2015-11-25 16:03:37 -0500 |
---|---|---|
committer | Michael Tokarev <mjt@tls.msk.ru> | 2015-12-04 09:39:55 +0300 |
commit | 624533e5a53e5df347e46f06408d15b9df5254f0 (patch) | |
tree | a5f33fcc5fb9791e2f7edb49ba3627289f9e5ecc /util | |
parent | 4c65fed8bdf96780735dbdb92a8bd0d6b6526cc3 (diff) | |
download | qemu-624533e5a53e5df347e46f06408d15b9df5254f0.zip qemu-624533e5a53e5df347e46f06408d15b9df5254f0.tar.gz qemu-624533e5a53e5df347e46f06408d15b9df5254f0.tar.bz2 |
util/id: fully allocate names table
Trivial: this array should be allocated to have ID_MAX entries always.
Otherwise if someone were to forget to expand this table, the assertion
in the id generator won't actually trigger; it will read junk data.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Jeff Cody <jcody@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Diffstat (limited to 'util')
-rw-r--r-- | util/id.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -29,7 +29,7 @@ bool id_wellformed(const char *id) #define ID_SPECIAL_CHAR '#' -static const char *const id_subsys_str[] = { +static const char *const id_subsys_str[ID_MAX] = { [ID_QDEV] = "qdev", [ID_BLOCK] = "block", }; @@ -53,7 +53,7 @@ char *id_generate(IdSubSystems id) static uint64_t id_counters[ID_MAX]; uint32_t rnd; - assert(id < ID_MAX); + assert(id < ARRAY_SIZE(id_subsys_str)); assert(id_subsys_str[id]); rnd = g_random_int_range(0, 100); |