Quote from: thesnat21 on March 13, 2019, 05:44:03 PM

Quote from: bible_pay on March 13, 2019, 04:52:59 PM

"StopCrashingBiblePay", that is such a Christian way to be Sun. It figures it was you as you are hiding behind being Christian but not acting like one.

Thanks for taking our chain down Sun; thats one thing Ive always disliked is hackers in prod.

I do think this should have been tested in testnet early on (special characters etc in the text fields). I am curious how this is different than tx message strings(Would this have a similar effect).

I cannot say it was malicious (with intent to crash the chain), I suspect it was only a matter of time before someone/something did this.

I do think we can add some tests to testnet next time around with this knowledge.

I made the assumption that our LIMITED_STRING was enough to prevent any effects of memory corruption, and for that I apologize to everyone.
In the future we wont base any production code on any untrusted user entered field ever again.

In my mind, the most effect the nickname could have had in the map was : Swear word (and we added code to remove those) and length, sorry guys.

In Evolution we will check every outside entry point in the smart contract processor.