allocate memory before calling blocksol

[dholth]

Description

This is a shorter fix for #38, with apologies to the original submitter.

[AndrewVallette]

Yes, this should solve the memory leak, but I'd like to see the allocation stay inside of blocksol. This ensures mem will be allocated if blocksol is ever called from a different location. (currently, it is not -- but I like to be safe :)

I would have opted for something closer to Kray-G's solution, except I would've assigned the dereferenced pointer to a local variable and used it in place of mem throughout the function. This would eliminate the need for so many dereferences.

[AndrewVallette]

The NULL check in blocksol combined with moving the allocation to the constructor looks good to me. 👍