[9fans] _threadmalloc() size>100000000; shouldn't be totalmalloc?

[adr <adr@SDF.ORG>]

/sys/src/libthread/lib.c

[...]
void*
_threadmalloc(long size, int z)
{
        void *m;

        m = malloc(size);
        if (m == nil)
                sysfatal("Malloc of size %ld failed: %r", size);
        setmalloctag(m, getcallerpc(&size));
        totalmalloc += size;
        if (size > 100000000) {
                fprint(2, "Malloc of size %ld, total %ld\n", size, totalmalloc);
                abort();
        }
        if (z)
                memset(m, 0, size);
        return m;
}
[...]

Shouldn't the if statement test the size of totalmalloc before the
abort? That size, 100M for internal allocations? It has to be
totalmalloc, doesn't it? If not this if statement should be after
testing the success of the malloc. Am I missing something?

adr.

------------------------------------------
9fans: 9fans
Permalink: https://9fans.topicbox.com/groups/9fans/Te1be8dc72738258d-M704d54262c512e9fce0f9fae
Delivery options: https://9fans.topicbox.com/groups/9fans/subscription