On Sep 17, 2009, at 10:21 AM, Harm van Tilborg wrote:
+ if (((vl.diskquota) && (bytes >= (vl.diskquota * 1000000))) || ((vl.maxmsgcount) && ((count >= vl.maxmsgcount))))

Maybe I'm doing too much embedded programming, but seeing the "* 1000000" makes me worry about overflow. If vl.diskquota is a 32-bit integer and you have a 5000MB quota, I think that the multiplication will overflow (5 billion can't be represented in a 32-bit int).

Safer to divide bytes by 1000000 and do the comparison. You'll get the same results.



Reply via email to