Barrapunto

1. El escribir la constante a la izquierda y la variable a la derecha es una "buena práctica" porque hace que los errores al confundir == con = salgan al compilar. Parece una tontería pero truquillos como este ahorran un montón de tiempo de depuración.
2. Ya, bueno, yo hablaba en general :P

3, 4, 5, 6: Ok.

Mmm, de vez en cuando se ven cosas interesantes entre tanto ruido como esta entrada y sus comentarios. Mi enhorabuena.

Al respecto del tema, un punto que no consigo ver es el del 100% de procesador consumido. Que hasta que no termine el hijo espere, fale, pero... ¿que esa espera sea activa? Where? Veo por ahí un waitpid(), que si no me falla la memoria bloquea a menos que especifiques alguna opción.

En general estoy bastante de acuerdo contigo, pero voy a poner yo algo de mi cosecha:

X+1) 0 no es lo mismo que NULL.

X+2) Las cosas pueden y suelen fallar. ¿Qué pasa si falla el exec()? Eso sí se puede llamar consumir CPU, y es solo un mal menor, porque el fork() debería estar protegido en la siguiente iteración al no existir la variable de entorno.

En resumen, muy mejorable, pero como dicen, nadie nace aprendido.