problema SDL carga imagenes

[calambrenet]

Tienes una manía fea ;) Pones unos identificadores muy parecidos entre distinto tipo y lleva a confusión.
Por ejmplo "SDL_Surface *imagen, *imagenes[IMAGENES];" estaría mejor "SDL_Surface *srf_imagen, *srf_imagenes[MAX_IMAGENES];" creo que así estaría mas sencillo y menos confuso.

Después para que imagenes (SDL_SURFACE) tenga un valor debe recogerlo de carga_imagenes. Seguro que si pones esto no te da error:

if(imagenes)
{
posicion.w = imagenes[PERSONAJE]->w; //segmentation fault
posicion.h = imagenes[PERSONAJE]->h;
}

Eso es porque el valor de imagenes es NULL
si la función carga_imagenes devuelve un puntero de SDL_SURFACE debes poner
imagenes = caga_imagenes(imagenes);

Creo que deberías plantearlo de otra manera.
Un saludo.