malloc funzt net?

hackbert

Hi Leute! Ich habe mir heute eine Funktion zum sicheren erstellen von Speicherbereichen für Strings geschrieben:

int safeCreateString(char* target, const char* source)
{
  	unsigned int length = strlen(source);
	target = (char*)malloc(length+1);
	if(!target)
		return -1;

	sprintf(target, "%s", source);
	return length;
}

Der als target angegebene String enthält danach aber nur wirres zeugs, nicht aber den string aus source...

interpreter

Deine Funktion wird nicht das machen, was sie eigentlich tun soll, da du die Adresse des reservierten Speicherbereichs in einem LOKALEN Zeiger speicherst, den es nach Beendigung der Funktion nicht mehr gibt.

Deine FUnktion sollte in etwa so aussehen (ungetestet):

int safeCreateString(char** target, const char* source) 
{ 
      unsigned int length = strlen(source); 
    *target = (char*)malloc(length+1); 
    if(!target) 
        return -1; 

    sprintf(*target, "%s", source); 
    return length; 
}

hackbert

Danke! Ich dachte immer pointer ist pointer...

interpreter

Nein, es gibt auch Pointer auf Pointer und Pointer auf Pointer auf Pointer usw

Shade Of Mine

Pointer ist Pointer - aber man kann auch eine Kopie von einem Pointer anlegen. Denn ein Pointer zeigt auf ein Objekt. Die Adresse dieses Objektes ist der Wert des Pointers. Und wenn wir diesen Wert (also die Adresse des Objektes) by value uebergeben, dann fuehrt ein aendern der Adresse natuerlich nicht zum gewuenschten Erfolg.

TheTester

unsigned int length = strlen(source);

seid ihr sicher das das so richtig ist?

bei diesem funktionskopf?

int safeCreateString(char** target, const char source

hmm....

bye

tt

SG1

Was sollte denn Deiner Meinung nach anders sein?

TheTester

Was sollte denn Deiner Meinung nach anders sein?

das die länge des char arrays mit als parameter übergeben wird

da ich davon ausgehe das sizeof(source) nur die größes eines pointers zurück gibt, würde man das vielleicht so char source[] übergeben sollte es gehen, da so eine komplette kopie des strings aufm stack gemacht wird

tschööö

tt

Shade Of Mine

char* und char[] sind in einem Funktionskopf das gleiche.
Deinen Punkt mit sizeof verstehe ich allerdings nicht.

TheTester

hmm..dann hab ich da sicher was verwechselt...ich für meinen teil bin da immer ziemlich misstrauisch und übergebe halt immer die anzahl mit über die parameterliste, vielleicht gewohnheit...irgendwelche gründe wirds sicher gehabt haben

tschööö

tt

Shade Of Mine

Performance Gruende hat es, wenn du die Laenge an die Funktion uebergibst - denn dann muss kein strlen() ausgefuehrt werden, wenn du die Laenge schon kennst. Und wenn du sie nicht kennst, kann man ja strlen() problemlos selber aufrufern.

Aber dennoch ist der obrige Code OK.

TheTester

jo der is ok..ich glaub ich hab strlen mit sizeof verwechselt

tschööö

tt

Aber dennoch ist der obrige Code OK.

Nope.

if(!target)
        return -1;

sollte wohl eher

if (!*target)
         return -1;

heißen

mfg

SG1

<edit>Muell</edit>

Shade Of Mine

gast schrieb:

Aber dennoch ist der obrige Code OK.

Nope.

Stimmt. Du hast recht.