Ошибка сегментации

[srs6270]

Здравствуйте,

дали задание написать код, который читает файл и затем записывает данные в буфер. Вроде написал, но не работает. Выяснил, что имеет место быть "ошибка сегментации". Попытки найти в Интернете дельный ответ увенчались провалом. Прошу вас помочь разобраться с кодом, ибо сам я в Си не особо силен.

#include <stdio.h> 
#include <string.h> 
#include <limits.h> 
 
int main(int argc, char *argv[]) { 
 
  FILE *fp; 
  char filename[128]; 
  char strings[USHRT_MAX][80]; 
  unsigned short cnt = 0; 
 
  strcpy(filename, argv[1]); 
 
  fp = fopen (filename, "r"); 
  if (fp == NULL) { 
    perror("Unable to open file: "); 
    return(-1); 
  } 
 
  while (fscanf(fp, "%s", strings[cnt++] ) != EOF); 
   
  fclose(fp); 
 
  return(0); 
}

[S1av0k]

Зачем в используете это:

strcpy(filename, argv[1]);

?

[Vago]

Он передаёт имя файла параметром при запуске. Ничуть, правда, не заботясь при этом о проверке длины имени...
...
Автор, массив на пять метров со стека снял бы, что-ли?..

[srs6270]

S1av0k, этот код:

strcpy(filename, argv[1]);

мне подсказали, но истинного значения не понимаю. Знаю, что он копирует аргумент в файл и все. Можно по другому сделать?
Vago, так лучше будет?

if(strlen(filename)>128)
{
printf("Filename is too long");
exit(0);
}

[Vago]

Ну да. Ну да... Сперва скопировали (возможно выйдя за пределы зарезервированной размерности), а потом проверяем длину!..

Гавная причина, по-прежнему, не в этом. Декларацию массива strings[USHRT_MAX][80] ЗА тело ф-ии вынеси, пожалуйста.

С типами памяти разберись. Си - это почти машинный язык. Нельзя на нём программировать и ничего при этом не знать об архитектуре машины (ох, пардон, компьютера).

[srs6270]

Vago, спасибо за наводку, почитаю и постараюсь понять.

[mmx358]

Доброго времени суток!

Не хочу создавать новую тему, поэтому пишу в этой. В ощем, уже несолько дней меня занимет одна проблема, причину которой я не могу обнаружить...

Итак, есть геттер класса CTTetromino (возвращает указатель на массив, TetroMap сам по себе и есть указатель на этот массив):

unsigned char*  CTTetromino::GetArray ( ) const
{
	return TetroMap;
}

Указатель на созданный статический объект этого класса (класса CTTetromino) передается в кач-ве аргумента в функцию (член-метод другого класса, однако, это не принципиально):

void CTGlass::PutTetroInTheGlass ( CTTetromino* Tetro )

Вот тело функции целиком:

void CTGlass::PutTetroInTheGlass ( CTTetromino* Tetro )
{
	unsigned char* TetroArray;

	int TetroX, TetroY;

	int X, Y;

	int i, j;

	int Tester;

	TetroArray = Tetro->GetArray ( );

	TetroX = Tetro->GetTetroMapX ( );
	TetroY = Tetro->GetTetroMapY ( );

	X = Tetro->GetPositionX ( );
	Y = Tetro->GetPositionY ( );

    for ( i = 0; i < TetroY; i++ )
    {
        for ( j = 0; j < TetroX; j++ )
        {
            //*( GlassArray + ( i + Y ) * TetroX + ( j + X ) ) = *( TetroArray + i * TetroX + j );
            //*( GlassArray + ( i + Y ) * TetroX + ( j + X ) ) = 0;
            Tester = *( TetroArray );
        }
    }
}

В теле объявляю указатель на массив:

unsigned char* TetroArray;

И инициализирую его с помощью вышеописанного геттера:

TetroArray = Tetro->GetArray ( );

Однако, при любой попытке обратиться к TetroArray:

Tester = *( TetroArray );

получаю ошибку сегментации.


Я просто понятия не имею, где я допустил оплошность... понятия не имею... не могу разобраться. Помогите, пожалуйста, я в тупике.

[Пепел Феникса]

покажите инициализацию массива вашего.

[mmx358]

Находится в конструкторе класса CTTetromino:

unsigned char* TetroMap;

int i, j; /// Counters for cycles

TetroMapX = ( TetroType == Fig_I ) ? 4 : ( ( TetroType == Fig_O ) ? 4 : 3 );
TetroMapY = ( TetroType == Fig_I ) ? 4 : 3;

TetroMap = ( unsigned char * ) calloc ( TetroMapX * TetroMapY, sizeof ( unsigned char ) );

for ( i = 0; i < TetroMapY; i++ )
for ( j = 0; j < TetroMapX; j++ )
{
    * ( TetroMap + i * TetroMapX + j ) = 0;
}

Вопрос все еще актуален.

[Hejerdal]

Ошибка происходит из-за выхода за пределы выделенной памяти. Проверьте, соответствуют ли значения переменных TetroMapX и TetroMapY в момент обращения к памяти в цикле таковым в момент выделения памяти под массив; правильно ли работают геттеры этих переменных. Используйте отладку!