Fix T49251: moving smoke domain with additional resolution causes crash.
authorAlexander Gavrilov <angavrilov@gmail.com>
Mon, 5 Sep 2016 12:50:12 +0000 (15:50 +0300)
committerAlexander Gavrilov <angavrilov@gmail.com>
Mon, 5 Sep 2016 12:50:12 +0000 (15:50 +0300)
commita31eca3fdd0d2e18b73f180b87a4c77ac5a8da78
tree4278934005fd85f74b7875a62459906c4dd64d91
parentc51cfbbc7f7be9b58f3354e0efe19aa488d984a9
Fix T49251: moving smoke domain with additional resolution causes crash.

This is a bug in the multithreaded task manager in negative value range.

The problem here is that if previter is unsigned, the comparison in the
return statement is unsigned, and works incorrectly if stop < 0 &&
iter >= 0. This in turn can happen if stop is close to 0, because this
code is designed to overrun the stop by chunk_size*num_threads as
the threads terminate.

This probably should go into 2.78 as it prevents a crash.
source/blender/blenlib/intern/task.c