Strings verketten in Pidgin-Plugin mit Programmiersprache C

[Dirtsa]

Ich hab die Unterhaltung noch im rss-Feed:

In C brauchst du halt Speicher, der groß genug ist, um die zusätzlichen Zeichen aufzunehmen. Der Speicher für "bla" ist genau 6 Byte groß (5 Zeichen + Nullbyte). strcat schreibt dahinter einfach weiter, wo entweder andere Dinge stehen oder wo kein weiterer Speicher ist. Wenn du Glück hast stürzt dein Programm einfach ab. Ansonsten provoziert es subtile Fehler, die an ganz anderer Stelle auftreten und nur schwer zu finden sind.


Faustregel: Benutze niemals strcat(). Die Funktion ist eine einzige Sicherheitslücke, weil sie die Größe des Zielspeichers nicht übergeben bekommt und daher nicht überprüfen kann, wo sie aufhören muss. strncat() wäre hier eine Alternative.

Ach Gottchen,
du hast natürlich Recht, ich danke dir vielmals!!

und ich habe strcat aus meinem Vokabular gestrichen, vielen Dank für diesen Hinweis!

Was sagst du dazu, fällt dir was "böses" auf?:

Code:

char* systemCall = (char*) malloc(1000);
*systemCall = *"";
char* userResource = (char*) purple_notify_user_info_entry_get_value(e);

strncat(systemCall, "explorer \"\\\\", strlen("explorer \" \\\\"));

strncat(systemCall, userResource, strlen(userResource));
strncat(systemCall, "\\C$\\\"", strlen("\\C$\\\""));

Lies mal die manpage zu strncat. Das will wissen, wieviel noch Platz in dem Puffer nach dem String noch ist