AK's weblog

Entries tagged as strdup

Related tags

c 23c3 assembler c++ contrapolice designbycontract dietlibc fun gcc git heap history introduction ndbm network overflow performance programming rant recursion security server stfl texttools trapdoor2 unix ada announcement ariane 5 army automated tests baconbird borland build management burning money callgraph cms code audit code coverage community compiler console continuous integration cruisecontrol dartlang debian debugging dot editor email embeddable feedreader fefe forced browsing free pascal freebsd fuzz gcov german glibc gnu pascal gnupg go gockel golang google hacking html http hudson i18n icu ide imap ipv6 javascript job lcov linux linuxwochen linz mobile mutt mutt-ng ncurses netbsd neted newsbeuter noos object-oriented obscure osx parser pascal patch management perl plan9 problem prototype pthread python quilt rails refactoring release rss ruby screencast scripting language shared library smtp stack terminal testing thread tpp twitter unicode video vienna vim wap web webdynamite websockets weird work x86 xml alcohol antigen arc argentina austria austrofascism autoconf automake beer berlin bsd bullshit christianity christmas citrus cmake digg discordianism eu fail football fundamentalism germany gnu going out gpl guard headache humanrights hype iconv indymedia jeopardy lawblog license lisp microsoft movie movies music newsgator one openbsd pkg-config politics pope programming language propaganda proxy quiz referee rootserver schwarzer scons sleep socialism solaris southpark ssh stallman terrorism toilet torture traffic travelling tv unemployment uno usa vcs wikipedia windows youtube

Friday, January 5. 2007

Das strdup-Desaster

Es soll ja alte Systeme geben, auf den die Funktion strdup() noch nicht vorhanden ist. Sun OS 3 ist angeblich so ein Kandidat. Nun, was macht man, wenn man diese Funktion nicht zur Verfuegung hat? Man implementiert sie selbst. Und hier ist das grosse Problem: kaum einer macht es richtig. Vor ein paar Wochen hab ich einen Patch an den fetchmail-Maintainer (gluecklicherweise hat ESR das abgegeben) deswegen geschickt. Und was sehe ich heute in der APR (die Apache Portable Runtime)? Dieses da:

#if !APR_HAVE_STRDUP
char *strdup(const char *str)
{
char *sdup;
size_t len = strlen(str) + 1;
sdup = (char *) malloc(len);
memcpy(sdup, str, len);
return sdup;
}
#endif

Zum Haare raufen. Also bin ich auf die Idee gekommen, einfach Google Codesearch zu verwenden, um eigene strdup()-Implementierungen zu finden. Tja, und das ging ziemlich schnell. Hier z.B..

char*
enca_strdup(const char *s) {
if (s == NULL)
return NULL;
else
return strcpy(enca_malloc(strlen(s) + 1), s);
}

enca_malloc() enthaelt uebrigens keinerlei Pruefung auf NULL, ausser ein assert(), d.h. wenn man mit CFLAGS=-DNDEBUG compiled, und in eine OOM-Situation geraet, fuehrt das zum gleichen Mist. Und auch MySQL sieht sehr vertrauenswuerdig aus:

#ifndef HAVE_STRDUP
char *
strdup(const char *s){
void *p2;
p2 = malloc(strlen(s)+1);
strcpy(p2, s);
return p2;
}
#endif

Und das in einer Datenbank von angeblicher Produktionsqualitaet. *ts*

Fazit: drei strdup()-Implementierungen begutachtet, keine einzige macht es richtig.

Posted by Andreas Krennmair at 14:35 | Comments (4) | Trackbacks (0)

Defined tags for this entry: c, programming, rant, strdup

Related entries by tags:

(Page 1 of 1, totaling 1 entries)