Разбор и манипулирование беззнаковыми символами в стиле C в C / C ++ - ошибка сегментации - PullRequest
Новая
       51

Разбор и манипулирование беззнаковыми символами в стиле C в C / C ++ - ошибка сегментации

0 голосов
/ 05 марта 2012

Обратите внимание, что для этого я использую компилятор C ++ (следовательно, приведение к вызовам функций calloc), но код по существу C.

По сути, у меня есть typedefна unsigned char, известный как viByte, который я использую для создания строкового буфера для анализа файла из двоичного файла (точнее, TGA-файла, но это не имеет значения).

I 'm написание основных функций для него прямо сейчас;append, prepend, new и т. д.

Проблема в том, что на первой итерации первого цикла в viByteBuf_Prepend я получаю ошибку сегментации.Мне нужно знать почему, именно потому, что это то, что может держать меня всю ночь без каких-либо указателей (каламбур).

Я также хотел бы знать, верны ли мои алгоритмы с точки зрения того, как буфер предварительно ожидает строку viByte.Например, у меня есть ощущение, что слишком большое использование memset может быть плохой идеей, и является ли мой формат printf для беззнакового символа правильным (у меня такое ощущение, что это не так, поскольку ничего не выводится на мойприставка).

Компиляция в GCC, Linux.

Ze Code 

#ifdef VI_BYTEBUF_DEBUG
void viByteBuf_TestPrepend( void )
{
    viByteBuf* buf = viByteBuf_New( 4 );

    buf->str = ( viByte* ) 0x1;

    printf(" Before viByteBuf_Prepend => %uc ", buf->str);

    viByteBuf_Prepend( buf, 3, ( viByte* ) 0x2 );

    printf(" After viByteBuf_Prepend => %uc ", buf->str);
}
#endif

viByteBuf* viByteBuf_New( unsigned int len )
{
    viByteBuf* buf = ( viByteBuf* ) calloc( sizeof( viByteBuf ), 1 );

    const int buflen = len + 1;

    buf->str = ( viByte* ) calloc( sizeof( viByte ), buflen );
    buf->len = buflen;

    buf->str[ buflen ] = '\0';

    return buf;
}

void viByteBuf_Prepend( viByteBuf* buf, unsigned int len, viByte* str )
{
    unsigned int pos, i;
    const unsigned int totallen = buf->len + len;
    viByteBuf* tmp = viByteBuf_New( totallen );
    viByte* strpos = buf->str;


    memset( tmp->str, 0, tmp->len );

    int index;

    for( i = 0; i < buf->len; ++i )
    {

       index = ( buf->len - i ) - 1;

       *strpos = buf->str[ 0 ];
       ++strpos;
    }

    memset( buf->str, 0, buf->len );

    printf( "%uc\n", buf->str );

    i = totallen;

    for ( pos = 0; pos < len; ++pos )
    {
        tmp->str[ pos ] = str[ pos ];
        tmp->str[ i ]   = buf->str[ i ];

        --i;
    }

    memset( buf->str, 0, buf->len );

    buf->len = tmp->len;

    memcpy( buf->str, tmp->str, tmp->len );

    viByteBuf_Free( tmp );

    //memset(  )
    //realloc( ( viByteBuf* ) buf, sizeof( viByteBuf ) * tmp->len );
}

Большое спасибо.

Обновление

Извините, я должен был явно опубликовать код, в котором лежит ошибка сегментации.Это прямо здесь: 

for( i = 0; i < buf->len; ++i )
{

   index = ( buf->len - i ) - 1;

   *strpos = buf->str[ 0 ]; //<--segmentation fault.
   ++strpos;
}

1 Ответ

0 голосов
/ 05 марта 2012

В вашем коде у вас есть buf->str[ buflen ] = '\0';, но вы выделяете место только для buflen. Я думаю, что вы имели в виду buf->str[ len ] = '\0';.

...