srg-imperial / SaBRe

Load-time selective binary rewriting
Other
152 stars 16 forks source link

Remove custom malloc #53

Closed andronat closed 4 years ago

andronat commented 4 years ago

Removed SaBRe's custom malloc as it is not used.

Also removed a couple of superfluous statements that are duplicates of here and here.

ccadar commented 4 years ago

@andronat Are you sure we want to remove it? I thought you're considering bringing it back. @parras What is your take?

parras commented 4 years ago

I think the idea is to replace this old, unreliable malloc with mimalloc -- which is fine. @andronat is that correct?

andronat commented 4 years ago

Yep! That’s exactly the plan as @parras says.

ccadar commented 4 years ago

Thanks, I'm then ok to remove this.