On Thu, Mar 27, 2014 at 05:09:41PM +0800, Fam Zheng wrote:
> + if (has_granularity) {
> + if (granularity & (granularity - 1)) {
> + error_setg(errp, "Granularity must be power of 2");
> + return;
> + }
> + } else {
> + granularity = 65536;
> + }util/hbitmap.c has: assert(granularity >= 0 && granularity < 64); Please make sure the argument is checked before we pass it down. We should never hit an assertion failure due to bad inputs. Stefan
