[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] util/id: fully allocate names table
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [PATCH v2] util/id: fully allocate names table |
Date: |
Sun, 29 Nov 2015 13:36:22 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.8.0 |
26.11.2015 00:03, John Snow wrote:
> 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.
Applied to -trivial as a bugfix, with a trivial commit description
cleanup, removing this part:
> v2: Fix the range assertion, too. Compare against the known actual size
> of the table instead of what it "should" be.
Thanks!
/mjt