OSDN Git Service

qom: assert integer does not overflow
authorMichael S. Tsirkin <mst@redhat.com>
Fri, 25 Feb 2022 13:40:27 +0000 (08:40 -0500)
committerMichael S. Tsirkin <mst@redhat.com>
Fri, 4 Mar 2022 10:40:58 +0000 (05:40 -0500)
commite36828787f2c35df64448f4142169ef5e193f5b3
tree65f5e383a5b5e9129abf075bd6f68752bfea03dc
parent6629bf78aac7e53f83fd0bcbdbe322e2302dfd1f
qom: assert integer does not overflow

QOM reference counting is not designed with an infinite amount of
references in mind, trying to take a reference in a loop without
dropping a reference will overflow the integer.

It is generally a symptom of a reference leak (a missing deref, commonly
as part of error handling - such as one fixed here:
https://lore.kernel.org/r/20220228095058.27899-1-sgarzare%40redhat.com ).

All this can lead to either freeing the object too early (memory
corruption) or never freeing it (memory leak).

If we happen to dereference at just the right time (when it's wrapping
around to 0), we might eventually assert when dereferencing, but the
real problem is an extra object_ref so let's assert there to make such
issues cleaner and easier to debug.

Some micro-benchmarking shows using fetch and add this is essentially
free on x86.

Since multiple threads could be incrementing in parallel, we assert
around INT_MAX to make sure none of these approach the wrap around
point: this way we get a memory leak and not a memory corruption, the
former is generally easier to debug.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
qom/object.c