I won't argue that using a heap allocator from within an ISR is a really bad idea but maybe it should be pointed out somewhere that doing so wouldn't be safe OR actually disable interrupts in here.