Jump to content

[1.298] CRoom::RoomUserPunish() input crash

Started by twostars, Feb 11 2012 01:07 PM

  • You cannot reply to this topic
6 replies to this topic

#1
twostars
Posted 11 February 2012 - 01:07 PM

twostars

    KO Guru

  • GateKeeper
  • Others:Donator
  • 4,479 posts
Yet another missing length check. Also fixed in 1.310 by default.

0048688F    ^\EB AC         JMP SHORT 0048683D

0048683D      83C4 18       ADD ESP,18
00486840    83F9 15         CMP ECX,15
00486843    0F83 19010000   JNB 00486962
00486849    40              INC EAX
0048684A    51              PUSH ECX
0048684B    EB 44           JMP SHORT 00486891

004868A1      83C4 0C       ADD ESP,0C

#2
twostars
Posted 18 February 2012 - 02:12 PM

twostars

    KO Guru

  • GateKeeper
  • Others:Donator
  • 4,479 posts
Updated to include missing stack cleanup. You'll want to repatch.

#3
lUizitO159
Posted 21 April 2012 - 08:39 PM

lUizitO159

    Post-a-haulic

  • Members
  • PipPipPipPip
  • 587 posts
what's this patch? what it does? fix any bug? or something? :D

#4
TheTompa
Posted 21 April 2012 - 09:23 PM

TheTompa

    Hacking Trainee

  • Donator
  • 73 posts

what's this patch? what it does? fix any bug? or something? :D

I think it's another way to crash the server, like this:
http://www.snoxd.net...3735#entry23735

#5
twostars
Posted 22 April 2012 - 03:30 AM

twostars

    KO Guru

  • GateKeeper
  • Others:Donator
  • 4,479 posts
Read the title. "Input crash".

It fixes a crash [method].

#6
lUizitO159
Posted 19 May 2012 - 07:35 PM

lUizitO159

    Post-a-haulic

  • Members
  • PipPipPipPip
  • 587 posts
ebenezer?

#7
twostars
Posted 19 May 2012 - 09:22 PM

twostars

    KO Guru

  • GateKeeper
  • Others:Donator
  • 4,479 posts
Yup. If I don't explicitly state it, assume [game] server.
Back to Assembly patches


  1. Snoxd
  2. → Board
  3. → Private server development
  4. → Knight Online private server development
  5. → Assembly patches
IPB skins by Skinbox