JoinedSenses / SM-SuperSprayHandler

Maintained version of Super Spray Handler sourcemod plugin by TheWreckingCrew6.
GNU General Public License v3.0
14 stars 7 forks source link

Non-standart symbols in nickname can crash server #3

Closed yarik2720 closed 4 years ago

yarik2720 commented 4 years ago

For example player had russian nickname: Бешанный енот ⭐⭐. If you watch on players spray server crashes:

SIGSEGV /SEGV_MAPERR accessing 0xbedb6000

Thread 0 (crashed):
   0: sourcemod.2.sdk2013.so!UserMessages::GetMessageIndex [ sm_stringhashmap.h:67 + 0x0 ] [ supersprayhandler.smx::.4480.StartMessageOne ]
      https://github.com/alliedmodders/sourcemod/blob/def6d0ff72e0d78c51007ee4b7fca1d83d566a77/public/sm_stringhashmap.h#L67

   1: jit_code_3627372544_1048576 + 0xce241 [ supersprayhandler.smx::.13716.Client_PrintKeyHintText ]
      eip: 0xd8423241  esp: 0xf8e02ac0  ebp: 0xf8e02ac8  

   2: jit_code_3627372544_1048576 + 0x82bc8 [ supersprayhandler.smx::CheckAllTraces ]
      eip: 0xd83d7bc8  esp: 0xf8e02ad0  ebp: 0xf8e02ad8  
JoinedSenses commented 4 years ago

Thanks for the report, just letting you know I have seen this but have yet to investigate. I have my suspicions.