Un buffer se me desborda o no sé que pasa!

[yogurete]

El siguiente código me peta, me da violación de segmento cuando lo que quiero comparar es cierto:

Código:

fgets(buffer,BUFFSIZE,stdin);
printf("Tengo en el buffer %s\n", buffer);

if (strcmp(buffer,"REGISTER \n")==0)
				{
				int i,j=0;
				for(i=9;i<19;i++)
					{
					if(buffer[i]!='\n') //ALMACENO NOMBRE USUARIO
						{
						usuari[j]=buffer[i];
						j++;
						}		
					}
                                 }

buffer estaba antes definido como:
#define BUFFSIZE 2048
char buffer[BUFFSIZE];


Alguna idea? Gracias

[MaxExtreme]

(Suponiendo que usuari apunta a una zona de memoria válida...)

El único fallo que veo es que nunca compruebas el '\0'. Te recuerdo que el terminador de una cadena no es '\n' sino '\0' y deberías comprobarle siempre, aparte de los demás que necesites.


Por cierto, haces algo extraño... ¿No es mejor simplificar así?

for(int i=9, int j=0;i<19&&buffer[i]!='\n'&&buffer[i]!='\0';i++,j++) usuari[j]=buffer[i];


O sin la j:

for(int i=9;i<19&&buffer[i]!='\n'&&buffer[i]!='\0';i++) usuari[i-9]=buffer[i];


O mejor aún:

int i=9;
while(i<19&&buffer[i]!='\n'&&buffer[i]!='\0') usuari[i-9]=buffer[i];


O incluso si quieres hacer el bobo todavía más:
int i=9;
while(i++<19&&buffer[i]!='\n'&&(usuari[i-9]=buffer[i])!='\0'); **

** Eso del ++ dentro puede no funcionar igual en los distintos compiladores, hay que tener cuidado.

Para que luego digan que sólo Perl tiene distintas maneras de hace runa misma cosa .

[Instru]

Mira, no se mucho del asunto, pero segun veo tu codigo no es a prueba de errores, osea no hace ninguna comprobacion antes de acceder a la memoria, alomejor esl ciclo es mas grande o alguna variable no tiene el suficiente numero de elementos para satisafacer el codigo, te recomiendo que no uses cosas asi, mejor usa codigoa prueba de errores.

Saludos