Comment 3 for bug 1818157

This just makes me want to cry. What is the purpose of aLength? A quick grep of the source code shows every call to StrNumCmp() has aLength set to INT_MAX. Why would you even do this? I cannot think of any case were dereferencing an invalid iterator is a good idea. I would prefer we fix this for 5.1 even if it pushes back the release.