hola tengo una duda sobre las funciones.
yo tengo una funcion llamada mifuncion() que devuekve -1 en caso de producirse un error.
mi duda: yo hago la siguiente asignacion:
   if(mifuncion() == -1){
      perror("Error");
   }

¿estaría bien el código o habria que asignarle previamente el valor devuelto por la funcion a una variable?:
     if((variable=mifuncion()) == -1){
        perror("Error");
     }

yo creo que con la primera opcion seria suficiente para saber si en la funcion se produce error.

¿alguien me aclara la duda?

Sí, la primera sería suficiente, pero la segunda como la tienes parece una repetición de la primera. Supongo que lo que querías poner es:

variable=mifuncion();
if((variable== -1){
    perror("Error");
}