Segmentation fault structs

Code:

func (sARGUMENTOS *argumentos, sGUARDA_RECTANGULOS *rects) {

        int erro=E_NOERR, i=0;
       
       

       
                while (rects!=NULL) {
                        rects++;
                        i++;
                }
       
       
                strcpy(rects->.Nome, argumentos[1].arg_num);
                rects->rect.x = 0;
                rects->rect.y = 0;
                rects->rect.tamx = argumentos[2].arg_num;
                rects->rect.tamy = argumentos[3].arg_num;
               
                if (strcmp(argumentos[4].arg_num, "n") == 0) {
                        rects->roda_rectangulo = NAO;
                }
                else {
                        rects->roda_rectangulo = SIM;
                }

               
                printf("%s %d %d %d %d", rects[i].Nome, rects[i].rect.x, rects[i].rect.y, rects[i].rect.tamx, rects[i].rect.tamy, rects[i].roda_rectangulo);
               


        return erro;


}

any suggestions?

Code:

                while (rects!=NULL) {
                        rects++;
                        i++;
                }

When does this loop stop? When rects is NULL. Thus, any time after this that you do rects->, you're accessing invalid memory, causing a seg fault.

Code:

       
                strcpy(rects->.Nome, argumentos[1].arg_num);

That's not valid syntax. You can't use the -> and the . together.

"When does this loop stop? When rects is NULL. Thus, any time after this that you do rects->, you're accessing invalid memory, causing a seg fault."

How to solve the problem?
objective is to insert the element at last position.


"That's not valid syntax. You can't use the -> and the . together. "

Yes, it's just ->

You're treating rects as though it's an array. You could simply pass in the current count of elements, then you would put your new data in rects[count].

an example with the code above?

Originally Posted by aalex

an example with the code above?

Since I seem to be on a tear about helpfiles today...

One of a programmer's most important skills is the ability to look stuff up.
Climb into your books, tutorials, help files, man pages, whatever and take a look at how arrays work...
See if you can come up with an answer on your own.

Then... if you get stuck (really stuck, not lazy stuck) post your code here and I'm sure some of us will help you sort it out.

Originally Posted by CommonTater

Since I seem to be on a tear about helpfiles today...

One of a programmer's most important skills is the ability to look stuff up.
Climb into your books, tutorials, help files, man pages, whatever and take a look at how arrays work...
See if you can come up with an answer on your own.

Then... if you get stuck (really stuck, not lazy stuck) post your code here and I'm sure some of us will help you sort it out.

I'm right back to it several hours. I think the problem is not laziness ...

Code:

int func_rect (sARGUMENTOS *argumentos, sGUARDA_RECTANGULOS *rects) {

	int erro=E_NOERR, i=0;
	
	
	/*verifica se o ultimo argumento do RECT é valido (s ou n)*/	
	
	if (strcmp(argumentos[4].arg_num, "n") != 0 && strcmp(argumentos[4].arg_num, "s") != 0) {
			erro = mensagem_de_erro(E_INVARGS);
	}
	
	else {
	
		while (rects!=NULL) {
			rects++;
			i++;
		}	
	
		strcpy(rects[i].Nome, argumentos[1].arg_num);
		rects[i].rect.x = 0;
		rects[i].rect.y = 0;
		rects[i].rect.tamx = argumentos[2].arg_num;
		rects[i].rect.tamy = argumentos[3].arg_num;
		
		if (strcmp(argumentos[4].arg_num, "n") == 0) {
			rects[i].roda_rectangulo = NAO_RODA_RECTANGULO;
		}
		else {
			rects[i].roda_rectangulo = RODA_RECTANGULO;
		}

		
		printf("%s %d %d %d %d", rects[i].Nome, rects[i].rect.x, rects[i].rect.y, rects[i].rect.tamx, rects[i].rect.tamy, rects[i].roda_rectangulo);

		rects=i+1;
		
	}

	return erro;


}

not already in segmentation fault

but not back to the program, gets stuck

Did you write the func_rect function? Do you know what a function parameter is? If so, add a third parameter to that function called count. If not, read your books and Google for some tutorials. Once you understand functions, add a third parameter called count. Then, ditch that while(rect != NULL) loop, since it doesn't work anyways, and just do rects[count] instead of rects[i].