Comment 296 for bug 195698

Your function CheckString leaks.
nsDialogParamBlock::GetString returns you a new copy of the string, so you must destroy it after you're finished comparing.

+ // We don't need to create a private copy because the
+ // param lives on lower stack frame.

I think GetString already gave you a private copy.