]> granicus.if.org Git - gc/commit
Workaround 'bad address arithmetic' static analysis tool false positive
authorIvan Maidanski <ivmai@mail.ru>
Sat, 17 Dec 2016 14:21:40 +0000 (17:21 +0300)
committerIvan Maidanski <ivmai@mail.ru>
Mon, 6 Feb 2017 17:30:16 +0000 (20:30 +0300)
commit8d56af65332ed54500899a74439bd30985d5b209
tree11e6c89a655a6446c905b778003a65852271c783
parent739e5e58ccd7d67e4ea92957833e11032de2dcbc
Workaround 'bad address arithmetic' static analysis tool false positive

The tool complains whether (alloc(size)+ofs) is intentional instead of
(alloc(size+ofs)).  In our case, it is a false alarm (the offset is
added to the result to align the allocation at HBLKSIZE boundary).

* os_dep.c [USE_WINALLOC && MSWIN32] (GC_win32_get_mem): Store result
of GlobalAlloc() to "result" local variable first (then, perform result
alignment in a standalone statement); add comment.
os_dep.c