2013-07-29 25 views
0

正如标题所说,无论何时我尝试复制字符串(使用我自己的字符串类型和库),都会出现段错误。根据调试器,它发生在第29行的源代码(其中调用了s_strcpy4()),它以“_dest [i] = _sour [i];”进入sstring.h中的行。 i在碰撞时似乎有不同的价值,但有时候会坚持一个值(不知道为什么)。截至写作它卡住在16896.使用指向uint64_t的指针在C中复制自定义字符串会导致段错误

任何人都知道我可以在代码中导致段错误?

由source.c

#include "sstring.h" 
#include <string.h> 
#include <stdio.h> 
#include <time.h> 

int main(void) { 
    s_string ss1, 
        ss2; 
    long long int start, end; 
    struct timespec time; 
    FILE *LoremIpsum; 

    s_init(&ss1, NULL, 65536); 
    s_init(&ss2, NULL, 65536); 

    LoremIpsum = fopen("Lorem ipsum", "r"); 
    if(LoremIpsum == NULL) { 
     perror("Error opening file "); 
     return 1; 
    } 

    fgets(ss2.string, 65536, LoremIpsum); 

    if(fclose(LoremIpsum) == EOF) { 
     perror("Error closing file "); 
     return 2; 
    } 

    s_strcpy4(&ss1, &ss2); 

    return 0; 
} 

sstring.h

#include <stdlib.h> 
#include <stdint.h> 

    enter code here 

typedef struct { 
    int length; 
    char *string; 
} s_string; 

s_string *s_init(s_string *str, char *array, size_t num) { 
    int i; 

    if(str == NULL) 
     str = malloc(sizeof(s_string)); 

    if(array == NULL) { 
     (*str).length = num; 

     if(num != 0) 
      (*str).string = malloc(num); 
    } else { 
     if(num == 0) { 
      (*str).string = NULL; 

      for(i = 0; array[i] != '\0'; i++) { 
       (*str).string = realloc((void *)(*str).string, i + 1); 
       (*str).string[i] = array[i]; 
      } 

      (*str).length = i; 
     } else { 
      (*str).string = (char *)malloc(num); 

      (*str).length = num; 

      for(i = 0; i < num; i++) 
       (*str).string[i] = array[i]; 
     } 
    } 

    return str; 
} 

s_string *s_strcpy4(s_string *__restrict__ destination, const s_string *__restrict__ source) { 
    int i; 
    long long *_dest = (long long *)(destination->string), 
         *_sour = (long long *)(source->string); 

    for(i = 0; i < source->length - 8; i+=8) 
     _dest[i] = _sour[i]; 

    for(; i < source->length; i++) 
     destination->string[i] = source->string[i]; 

    destination->length = i; 

    return destination; 
} 
+0

's_init(p,NULL,0)''不初始化'p-> string'。我不确定它是*问题,但它是一个*问题。 – Casey

+0

谢谢,没有意识到。现在它在这种情况下将p-> string设置为NULL。 –

回答

1

_dest [I] = _sour [I]; 应该是 _dest [i/8] = _sour [i/8];

或类似的东西。

代码还有很多其他的错误。例如,在需要正确对齐指针的体系结构上,尝试访问八个字符组中不能以八位数的倍数开始的字符将导致总线错误。

我假设你是在做这个练习,而不是在生产中使用。只使用memcpy()

+0

很高兴知道我的代码有什么问题,而不仅仅是因为它有很多错误。 –

0
size_t const LLsize = sizeof(long long); 
for(i = 0; i < source->length/LLsize; ++i) 
    _dest[i] = _sour[i]; 

for(i = i*LLsize; i < source->length; ++i) 
    destination->string[i] = source->string[i];