Da gibt es wirklich sehr viele Kritikpunkte. Ich hab auch noch ein paar:

- sizeof/strlen/sonstige.... zählst du noch oder programmierst du schon?
- Die Berechnung der Funktionslänge ist echt sehr schlecht und ungenau. Am besten ist es, wenn man am Ende der Funktion eine Markierung reinmacht und diese dann sucht z.B.
Code:
	__asm {
		int 0x80
		int 0x80
		int 0x80
	}
- Api Adresse als Parameter übergeben ist nicht gerade gut... das schwierige an sowas ist doch immer, wie bekomme ich meinen Code so portable wie Shellcode und das Problem hast du schlecht gelöst.

Fazit: Unbrauchbarer Tutorial, für ein Anfänger zu schlecht und für ein Profi eine Lachnummer.