---

Er der nogen der kan forklare mig hvad jeg gør galt i følgende kode stump?
....
csWriteLink("1ID");
....
int csWriteLink(char *string)
{
char *checksumString;
char *completeString;
printf("Writing with cs.\n");
sprintf(checksumString,"\x02%s\x03",string); /*Her får jeg segment fejl*/
.....
}

---

Hans Hvelplund Odborg skrev:

>Er der nogen der kan forklare mig hvad jeg gør galt i følgende kode stump?

Har du allokeret hukommelse til de to pointere?

>...
> csWriteLink("1ID");
>...
>int csWriteLink(char *string)
>{
> char *checksumString;
> char *completeString;

checksumString = malloc (GodtMedHukommelse);
completeString = malloc (MasserAfHukommelse);

> printf("Writing with cs.\n");
> sprintf(checksumString,"\x02%s\x03",string); /*Her får jeg segment fejl*/
> .....
>}

--
Bertel
http://lundhansen.dk/bertel/   FIDUSO: http://fiduso.dk/

---

Bertel Lund Hansen <nospamto@lundhansen.dk> skrev:
>Har du allokeret hukommelse til de to pointere?

(Teknisk detalje: svaret hertil er ja, for de to pointere er jo ganske
almindelige lokale variabler - men de *peger* ikke på noget allokeret
hukommelse.)

>>int csWriteLink(char *string)
>>{
>> char *checksumString;
>
> checksumString = malloc (GodtMedHukommelse);

Dårlig ide. Lad være med at gætte. Brug i stedet kendskabet til den
fornødne plads:

checksumString = malloc(strlen(string)+3); /* \2 string \3 \0 */

>> sprintf(checksumString,"\x02%s\x03",string); /*Her får jeg segment fejl*/

--
Richard Flamsholt
richard@flamsholt.dk - www.richard.flamsholt.dk

---

Richard Flamsholt <richard@flamsholt.dk> skrev:

> > checksumString = malloc (GodtMedHukommelse);
>
> Dårlig ide. Lad være med at gætte. Brug i stedet kendskabet til den
> fornødne plads:
>
> checksumString = malloc(strlen(string)+3); /* \2 string \3 \0 */

Eller brug ellers i det mindste snprintf() til at sikre, at
GodtMedHukommelse rent faktisk var stor nok.

--
Regards, Anders

....if a Microsoft product fails, who do you sue?

---

Richard Flamsholt skrev:

>> checksumString = malloc (GodtMedHukommelse);

>Dårlig ide. Lad være med at gætte.

Helt enig. Jeg mente bare at de nærmere detaljer overlodes til
læseren, men det var selvfølgelig ikke skrevet så klart.

--
Bertel
http://lundhansen.dk/bertel/   FIDUSO: http://fiduso.dk/

---

Richard Flamsholt <richard@flamsholt.dk> writes:

> checksumString = malloc(strlen(string)+3); /* \2 string \3 \0 */

Og husk at kalde

free (checksumString)

når du forlader funktionen, for at rydde op. Alternativt kan du
erklære checksumString således:

char checksumString[strlen (string) + 3];

Så sker oprydningen helt automatisk. Det kræver dog at din kompiler
er ny (eller GNU) nok, da ældre C versioner ikke understøtter
ovenstående.

---

> Er der nogen der kan forklare mig hvad jeg gør galt i følgende kode stump?
> ...
> csWriteLink("1ID");
> ...
> int csWriteLink(char *string)
> {
> char *checksumString;
> char *completeString;
> printf("Writing with cs.\n");
> sprintf(checksumString,"\x02%s\x03",string); /*Her får jeg segment
fejl*/
> .....
> }

Som Bertel allerede har nævnt skal du huske at allokere hukommelse til dine
char * 's.
sprintf() og de andre strenge-manipulationskald allokerer ikke selv den
hukommelse der skal bruges. Da jeg kodede mit første C undrede det mig også,
men i længden er det klart bedre selv at styre hukommelses-allokeringer.

---

Hans Hvelplund Odborg <hhod00@control.auc.dk> skrev:
> Er der nogen der kan forklare mig hvad jeg gør galt i følgende kode stump?
> ...
> csWriteLink("1ID");
> ...
> int csWriteLink(char *string)
> {
> char *checksumString;
> char *completeString;
> printf("Writing with cs.\n");
> sprintf(checksumString,"\x02%s\x03",string); /*Her får jeg segment fejl*/
> .....
> }

En anden ting (udover hvad andre allerede har nævnt) er at du burde
erklære string som "const char *". Det bør erklæres at funktionen
ikke ændrer hvad string peger på fordi:

- Det giver mere information til et menneske som læser koder og
prøver på at finde ud hvad der sker.

- Det forøger oversætteren muligheder for at komme med
fejlmeldninger hvis der alligevel rettes i konstante data.

- Det forøger oversætteren muligheder for optimeringer.

- Funktionen kaldes med en konstant streng som ikke må ændres.