Comment 2 for bug 1106116

Revision history for this message
Thorsten Glaser (mirabilos) wrote :

mksh R42 contains code protected by #ifdef DEBUG_LEAKS (enabled #ifdef DEBUG) to handle this.

For Android, defining DEBUG_LEAKS (but not DEBUG, never in production!) is probably sensible, but *only* after this has been tested to ⓐ not crash, ⓑ not leak, and ⓒ the exclusion of mksh in the malloc checker is reversed (though ⓒ can be done after defining DEBUG_LEAKS). I will try to set up a new VM with AOSP and submit an update of mksh soonish (this time *buntu 10.04 as building AOSP on Debian is nigh impossible and changes every time I try).