#206 Memory leak in HashMgr::remove(), remove_forbidden_flag()

[Justin Lebar]

HashMgr::remove() and HashMgr::remove_forbidden_flag both set dp->astr without first freeing it.

Unless I'm missing something pretty basic here, this is a memory leak.

I'd be happy to write a patch, if you'd like. I think the fix is as easy as adding two free() calls.

[Justin Lebar]

This patch is untested because we don't end up calling these functions in Mozilla.