Spot the bug

Microsoft Security Developer Center , http://msdn.microsoft.com/security/ , har börjat med nåt som kallas "spot the bug", jag tror att ni fattar vad det går ut på så här kommer koden. Första exemplet är ganska enkelt så de flesta bör hitta felet.

#define MAX (50)
char szDest[MAX];
strncpy(szDest,pszSrc,MAX);
pszDest[MAX] = '\0';

Får hålla tråden uppdaterad med fler exempel när det kommer. Om ni har nån egen klurig och buggig kod kan ni ju paste:a den här också. Vi håller oss till C/C++ i denna tråd så inget annat tack.

"pszDest" är ju bara ett stavfel, skulle man ju hittat vid kompileringen, man får väl anta att pszSrc är en null-terminerad sträng då den inte finns deklarerad i koden. Handlar ju om att hitta buggar och inte kompileringsspel ..

Kopierar en sträng till en array. Sista raden ska ju se till att strängen blir null-terminerad men koden skriver utanför arrayen, då arrayen går ifrån 0-(MAX-1). Är egentligen ett ganska jobbigt fel då man inte alltid märker att felet uppstår.

strncpy() = string copy .. kopierar en sträng .. max n st bokstäver, bör användas istället för strcpy() ..
szDest = en array
pszSrc = en pekare till en null (zero) terminerad sträng. Dvs. en sträng som slutar med en byte med värdet 0..
pszDest = szDest = stavfel ..

[EDIT] 'p' i variabelnamnet betyder "pointer" .. dvs. pekare ..
's' = string
'z' = zero-terminated, dvs. null-terminerad