Komme mit MD5 nicht klar.

Masterman · 2. Januar 2013

Hallo leute. Ich will mein GF Hashen also mit MD5 das passwort verschlüsseln nur irgendwas läuft da immerwieder falsch ich weiß aber nicht was :X

Hier das steht in der Userfile

SQL

Key=C9F3795EF24A9355BB016518114Level=1

ich versteh nicht warum level auf der gleichen zeile wie passwort ist?
hier der scriptteil

public OnPlayerRegister(playerid, password[])
{
if(IsPlayerConnected(playerid))
{
new string3[64];
new playername3[MAX_PLAYER_NAME];
GetPlayerName(playerid, playername3, sizeof(playername3));
format(string3, sizeof(string3), "/Accounts/%s.ini", playername3);
new File: hFile = fopen(string3, io_write);
if (hFile)
{
strmid(PlayerInfo[playerid][pKey], password, 0, strlen(password), 255);
new var[128];
format(var, 128, "Key=%s\n", MD5_Hash(PlayerInfo[playerid][pKey]));fwrite(hFile, var);

xinnu · 2. Januar 2013

Mach es mal mit Notpad++ auf, das ist ein fehler, ausgelesen wird es ja richtig oder?

Masterman · 2. Januar 2013

ich habe das mit notepad++ geöffnet und ja eigentlich schon

public OnPlayerLogin(playerid,password[])
{
new string2[128];
new playername2[MAX_PLAYER_NAME];
GetPlayerName(playerid, playername2, sizeof(playername2));
format(string2, sizeof(string2), "/Accounts/%s.ini", playername2);
new File: UserFile = fopen(string2, io_read);
if ( UserFile )
{
new PassData[256];
new keytmp[256], valtmp[256];
fread( UserFile , PassData , sizeof( PassData ) );
keytmp = ini_GetKey( PassData );
if( strcmp( keytmp , "Key" , true ) == 0 )
{
valtmp = ini_GetValue( PassData );
strmid(PlayerInfo[playerid][pKey], valtmp, 0, strlen(valtmp)-1, 255);
}
//if(strcmp(PlayerInfo[playerid][pKey],MD5_Hash(password), true ) == 0 )
if(strcmp(MD5_Hash(password),dini_Get(string2,"Key")) == 0)
{

Campbell · 2. Januar 2013

Verwende kein MD5, es gibt weit bessere Varianten Passwörter zu hashen.
Verschlüsseln ist nicht das selbe wie hashen. MD5 ist eine Hashfunktion - keine Verschlüsselung.
Dein Problem muss beim Schreiben entstehen, ich sehe jedoch kein "Level" unter OnPlayerRegister() - bitte etwas mehr Quellcode zeigen.

Masterman · 2. Januar 2013

Campbell was könntest du denn für Verschlüsselungen empfehlen also ich hätte Whirlpool im auge.

Hier ein weiterer teil onplayerregister

public OnPlayerRegister(playerid, password[])
{
if(IsPlayerConnected(playerid))
{
new string3[64];
new playername3[MAX_PLAYER_NAME];
GetPlayerName(playerid, playername3, sizeof(playername3));
format(string3, sizeof(string3), "/Accounts/%s.ini", playername3);
new File: hFile = fopen(string3, io_write);
if (hFile)
{
strmid(PlayerInfo[playerid][pKey], password, 0, strlen(password), 255);
new var[32];
format(var, 32, "Key=%s\n", MD5_Hash(PlayerInfo[playerid][pKey]));fwrite(hFile, var);
PlayerInfo[playerid][pCash] = GetPlayerMoney(playerid);
format(var, 32, "Level=%d\n",PlayerInfo[playerid][pLevel]);fwrite(hFile, var);
format(var, 32, "AdminLevel=%d\n",PlayerInfo[playerid][pAdmin]);fwrite(hFile, var);
format(var, 32, "Supporter=%d\n",PlayerInfo[playerid][pSupporter]);fwrite(hFile, var);
format(var, 32, "DonateRank=%d\n",PlayerInfo[playerid][pDonateRank]);fwrite(hFile, var);
format(var, 32, "pFirstJob=%d\n",PlayerInfo[playerid][pFirstJob]);fwrite(hFile, var);
format(var, 32, "pHotel=%d\n",PlayerInfo[playerid][pHotel]);fwrite(hFile, var);
format(var, 32, "ConnectedTime=%d\n",PlayerInfo[playerid][pConnectTime]);fwrite(hFile, var);

Ditti · 2. Januar 2013

Mal so als Tipp: nur Linux stellt \n als Zeilenumbruch dar, Windows nimmt \r\n. Im Notepad wird es also als eine Zeile dargestellt. Notepad++ sollte jedoch mit \n klar kommen. Und soweit ich weiß, wird bei den Files eben auch immer \n genommen.
Du kannst aber mal schauen, ob du auch problemlos \r\n verwenden kannst.

Grex · 2. Januar 2013

Zitat von Campbell ©

Verwende kein MD5, es gibt weit bessere Varianten Passwörter zu hashen.

Warum nicht MD5??

Campbell · 2. Januar 2013

Zitat von Grex

Warum nicht MD5??

Wikipedia:

Zitat

In 1996, a flaw was found with the design of MD5, and while it was not a clearly fatal weakness, cryptographers began recommending the use of other algorithms, such as SHA-1—which has since been found to be vulnerable as well. In 2004, more serious flaws were discovered in MD5, making further use of the algorithm for security purposes questionable—specifically, a group of researchers described how to create a pair of files that share the same MD5 checksum.[4][5] Further advances were made in breaking MD5 in 2005, 2006, and 2007.[6] In December 2008, a group of researchers used this technique to fake SSL certificate validity,[7][8] and US-CERT now says that MD5 "should be considered cryptographically broken and unsuitable for further use"

Zitat

Sie gilt inzwischen nicht mehr als sicher, da es mit überschaubarem Aufwand möglich ist, unterschiedliche Nachrichten zu erzeugen, die dieselbe MD5-Prüfsumme aufweisen.

Ebenfalls gibt es im Internet viele Datenbanken mit unbeschreiblich vielen MD5 Hashs, womit es leicht wird, das dazugehörige Passwort herauszusuchen. Whirlpool ist die wohl gerade für Pawno beste Hashfunktion, die zu bieten ist. Bestenfalls wäre die Speicherung von Passwörtern über Umwege (PHP etc.), da diese SHA-Hashfunktionen verfügen und auch einen Salt einbauen können.

maddin · 2. Januar 2013

MD5 reicht für nen SAMP Server völlig aus. Als ob irgendwer scharf auf Passwörter von irgendeinem Server wäre... tz.

Grex · 2. Januar 2013

Wenn man PWs wie 1234567 hat oder apfelbaum usw. ist das schon klar.
Wenn man aber ein PW wie dieses hat:
a!g?H%.-369 hat ist das schon etwas anderes und es steht bestimmt nicht in einer md5 Datenbank, höchtens in einer Rainbow Table, aber die sind mehrere 100GB groß und dauert auch ewig.

Logan · 2. Januar 2013

Zitat von Grex

Wenn man PWs wie 1234567 hat oder apfelbaum usw. ist das schon klar.
Wenn man aber ein PW wie dieses hat:
a!g?H%.-369 hat ist das schon etwas anderes und es steht bestimmt nicht in einer md5 Datenbank, höchtens in einer Rainbow Table, aber die sind mehrere 100GB groß und dauert auch ewig.

Ich glaube aber auch nicht, dass du ein PW wie a!g?H%.-369 auf einem SAMP Server wählst.

MfG

Campbell · 2. Januar 2013

Macht es ordentlich oder lasst es sein. Ob ihr nun Weg 1 oder 2 nehmt ist ja wohl egal. Hier kommt es darauf an was sinnvoller ist und MD5 ist nun mal garnicht sinnvoll.

Masterman · 2. Januar 2013

Könnte mir nun wer helfen wie ich das mit GF mache ?

Campbell · 2. Januar 2013

Zitat von Masterman

Könnte mir nun wer helfen wie ich das mit GF mache ?

Siehe Beitrag Ditti und hier: http://de.wikipedia.org/wiki/Z…ierung_des_Zeilenumbruchs

Masterman · 2. Januar 2013

So ich habs probiert aber das passwort bleibt so wie es ist ???

onplayerlogin
new Buff[129], str[129];
strmid(str,dini_Get(string2,"Key"),0,128,255);
WP_Hash(Buff,sizeof(Buff),str);
if(strcmp(Buff,str, true ))

onplayerregister

new Buff[129], str[129];
strmid(str,password,0,strlen(password),255);
WP_Hash(Buff,sizeof(Buff),str);

Dudalus · 2. Januar 2013

In Zeiten wo PHP Whirlpool und SHA unterstützt ist das echt kein grosses Ding mehr, KEIN MD5 zu verwenden

Campbell · 2. Januar 2013

Zitat

So ich habs probiert aber das passwort bleibt so wie es ist ???

Was bleibt wie es ist und wo liegt das Problem?

Masterman · 2. Januar 2013

ich habe z.B. beim passwort 12345 eingegeben und es steht in der datei auch als 12345 obwohl
onplayerlogin

new Buff[129], str[129];
strmid(str,dini_Get(string2,"Key"),0,128,255);
WP_Hash(Buff,sizeof(Buff),str);
if(strcmp(Buff,str, true ))

onplayerregister

new Buff[129], str[129];
strmid(str,password,0,strlen(password),255);
WP_Hash(Buff,sizeof(Buff),str);

Campbell · 3. Januar 2013

Deine Codeausschnitte sind für die Tonne. Du erstellst hier zwar einen Hash, es hat jedoch nichts mit der Speicherung des Hashs zu tun.

Masterman · 3. Januar 2013

public OnPlayerRegister(playerid, password[])
{
if(IsPlayerConnected(playerid))
{
new string3[64];
new playername3[MAX_PLAYER_NAME];
GetPlayerName(playerid, playername3, sizeof(playername3));
format(string3, sizeof(string3), "/Accounts/%s.ini", playername3);
new File: hFile = fopen(string3, io_write);
if (hFile)
{
new Buff[129], str[129];
strmid(str,password,0,strlen(password),255);
WP_Hash(Buff,sizeof(Buff),str);
new var[32];
format(var, 32, "Key=%s\n", str);fwrite(hFile, var);

public OnPlayerLogin(playerid,password[])
{
new string2[128];
new playername2[MAX_PLAYER_NAME];
GetPlayerName(playerid, playername2, sizeof(playername2));
format(string2, sizeof(string2), "/Accounts/%s.ini", playername2);
new File: UserFile = fopen(string2, io_read);
if ( UserFile )
{
new PassData[256];
new keytmp[256], valtmp[256];
fread( UserFile , PassData , sizeof( PassData ) );
keytmp = ini_GetKey( PassData );
if( strcmp( keytmp , "Key" , true ) == 0 )
{
valtmp = ini_GetValue( PassData );
strmid(PlayerInfo[playerid][pKey], valtmp, 0, strlen(valtmp)-1, 255);
}
new Buff[129], str[129];
strmid(str,dini_Get(string2,"Key"),0,128,255);
WP_Hash(Buff,sizeof(Buff),str);
if(strcmp(Buff,str, true ))