[?RELEASE?]Source(ugly, untested) of /dice fix

[TheMarv :<]

Heyho,

ich weiß nicht obs funtzt oder nicht, und ja ich weiß die Source ist hässlich,
ich hatte diese ganzen Threads von wegen Fix für die /dice Funktion gefunden und habe dann einfach mal am Handy eine (wahrscheinlich buggy) Funktion zum Überschreiben geschrieben.

Es geht an sich um die Funktion number_ex...wenn jemand sieht das es nicht funktioniert, würde ich ihn doch liebend gerne beten es zu fixen und hier im Thread zu posten, das wäre sehr nett ^-^


Naja hier habt ihr die Source:

 Spoiler

PHP Code:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <stdbool.h>
#include <dlfcn.h>
#include <sys/mman.h>
#include <inttypes.h>
#include "ddetour.h"

void __attribute__ ((constructor)) load(void);

int my_number_ex(int arg1, int arg2)
{
    char* error=(char)"no error";
    long first = (long)arg1;
    long second = (long)arg2;
    
    if(first > second)
    {
        long tmp_second = second;
        second = first;
        first = tmp_second;
    }
    if(first == second)
    {
        error = "Your arguments are equal!";
    }
    if(first <= 0 || second <= 0)
    {
        error = "Someone tried to down your server!\nGo thank TheMarv :P";
    }
    long the_mod = (second + 1)-first;
    if(sizeof(error)!=8)
    {
        printf("%s caused by: %d %d\n", error, first, second)
        return 0;
    }
    return first + thecore_random() % the_mod;
}

void load(void)
{
    printf("By TheMarv\n");
    number_ex=(int (*)(int, char *argv[]))detour((void*)0x08315F80,(void *)my_numer_ex, DETOUR_LEN_AUTO);
} 


ob es funktioniert weiß ich wie gesagt nicht, ich hoffe schon xD

MfG

[.Bloddy]

work 100%

[TheMarv :<]

Originally Posted by .Bloddy

work 100%

are you sure? are you really, really sure?

[Mashkin]

Seems legit because you first need to tell the compiler what thecore_random points to (you cannot call functions by name without introducing their addresses).

The function is roughly the same - except that the error handling looks better to me.
The reference to sys_err is missing though.

[TheMarv :<]

Originally Posted by Mashkin

Seems legit because you first need to tell the compiler what thecore_random points to (you cannot call functions by name without introducing their addresses).

The function is roughly the same - except that the error handling looks better to me.
The reference to sys_err is missing though.

i knew i forgot something....could you tell me the address of the thecore_random function?

[Lord iRemix]

Ein Tpp:

Wenn bei einer if-Abfrage nur eine Zeile folgt, musst du die geschweiften Klammern nicht machen.
Das Gleiche auch bei Schleifen etc.

So zB:


Spart Zeit am Handy

[iMer]

Originally Posted by .Remix.

Ein Tpp:

Wenn bei einer if-Abfrage nur eine Zeile folgt, musst du die geschweiften Klammern nicht machen.
Das Gleiche auch bei Schleifen etc.

So zB:


Spart Zeit am Handy

Sollte man aber trozdem machen, da es zur lesbarkeit beiträgt.

[TheMarv :<]

Originally Posted by .Remix.

Ein Tpp:

Wenn bei einer if-Abfrage nur eine Zeile folgt, musst du die geschweiften Klammern nicht machen.
Das Gleiche auch bei Schleifen etc.

So zB:


Spart Zeit am Handy

jo schon, aber ist nicht so meine Art :P
Ab und zu mach ich es, meistens vergesse ich es einfach xD

[xCPx]

Wieso zur hölle schaut eig niemand danach, was fürn rückgabewert number_ex eigentlich hat?
Selbst die Lösung von Luca ist falsch, da er auch nicht drauf geachtet hat.
Hier mal meine Variante (kp ob gut, aber immerhin mitm richtigen Rückgabe-Wert)

in den header:

unsigned int (*number_ex)(int arg1, int arg2) = (unsigned int(*)(int,int))0x08439620;
int (*thecore_random)() = (int(*)())0x08439610;

in die main.cpp

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <math.h>
#include <stdbool.h>
#include <dlfcn.h>
#include <sys/mman.h>
#include <inttypes.h>
#include "utils/ddetour.h"
#include "gamefunctions.h"


void __attribute__ ((constructor)) my_load(void);

unsigned int fix_number_ex(int a1, int a2)
{
	int temp;
	if(a1 <0)
	{
		a1 += -(a1*2);
	}
	if(a2 <0)
	{
		a2 += -(a2*2);
	}
	if(a1 > a2)
	{
		temp = a1;
		a1 = a2;
		a2 = temp;
	}
	temp = (a2 +1 - a1)
	if(temp <= 0)
	{
		temp = 1;
	}
	return a1 + thecore_random() % temp;
}

void my_load(void)
{
	srand(time(0));
	number_ex = (unsigned int (*)(int, int))detour((unsigned int*)0x08439620,(unsigned int *)fix_number_ex, DETOUR_LEN_AUTO, 0);
}

[TheMarv :<]

Originally Posted by xCPx

Wieso zur hölle schaut eig niemand danach, was fürn rückgabewert number_ex eigentlich hat?
Selbst die Lösung von Luca ist falsch, da er auch nicht drauf geachtet hat.
Hier mal meine Variante (kp ob gut, aber immerhin mitm richtigen Rückgabe-Wert)

in den header:

unsigned int (*number_ex)(int arg1, int arg2) = (unsigned int(*)(int,int))0x08439620;
int (*thecore_random)() = (int(*)())0x08439610;

in die main.cpp

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <math.h>
#include <stdbool.h>
#include <dlfcn.h>
#include <sys/mman.h>
#include <inttypes.h>
#include "utils/ddetour.h"
#include "gamefunctions.h"


void __attribute__ ((constructor)) my_load(void);

unsigned int fix_number_ex(int a1, int a2)
{
	int temp;
	if(a1 <0)
	{
		a1 += -(a1*2);
	}
	if(a2 <0)
	{
		a2 += -(a2*2);
	}
	if(a1 > a2)
	{
		temp = a1;
		a1 = a2;
		a2 = temp;
	}
	temp = (a2 +1 - a1)
	if(temp <= 0)
	{
		temp = 1;
	}
	return a1 + thecore_random() % temp;
}

void my_load(void)
{
	srand(time(0));
	number_ex = (unsigned int (*)(int, int))detour((unsigned int*)0x08439620,(unsigned int *)fix_number_ex, DETOUR_LEN_AUTO, 0);
}

Und was wenn beide Werte gleich sind? :P

[xCPx]

Originally Posted by TheMarv :<

Und was wenn beide Werte gleich sind? :P

Haste die exception doch drin:
temp = (a2 +1 - a1)


Wenn beide gleich sind ergibt das 1

Und sollte doch 0 rauskommen, so ist es danach durch:
if(temp <= 0)
{
temp = 1;
}
Wd 1

also von daher sollte das alles gefixxt sein.

[TheMarv :<]

Originally Posted by xCPx

Haste die exception doch drin:
temp = (a2 +1 - a1)


Wenn beide gleich sind ergibt das 1

Und sollte doch 0 rauskommen, so ist es danach durch:
if(temp <= 0)
{
temp = 1;
}
Wd 1

also von daher sollte das alles gefixxt sein.

achso ja, hatte nur drüber geschaut gehabt xD
naja aber was solls, es werden sowieso ein großteil der server nicht einbauen :P

[Computerfreek]

Gut, dass es uns ohne die ddetour.h was nützt.

[TheMarv :<]

Originally Posted by Computerfreek

Gut, dass es uns ohne die ddetour.h was nützt.

die gibts beim thread wie man .so extensions macht mit dabei....

[Mi4uric3]

Originally Posted by Computerfreek

Gut, dass es uns ohne die ddetour.h was nützt.

Eine Detour-Funktion zu schreiben ist nicht sonderlich kompliziert..

Pseudocode

void detour(void* src, void* dest) {
	mprotect von src mit size 5
	unsigned char array mit größe 5 erstellen
	array[0] = E9
	array[1] = dest - src - 5
	memcpy von array nach src mit der länge 5
}