2013-06-03 39 views
1

我一直在努力与这个bug一段时间,我无法弄清楚什么是错的。下面的代码:打开一个文件给sYSMALLOc断言失败

//the code for the function that is being called 
//charset is a const char[] consisting of 91 characters 
//charset_size is 91 
void set_sequence(char keyword[], int keyword_size){ 

    sequence = malloc(keyword_size); 

    int i = 0, j = 0; 

    for(i = 0; i < keyword_size; i++){ 

     for(j = 0; j < charset_size; j++){ 

      if(keyword[i] == charset[j]){ 

       sequence[i] = j; 

      } 

     } 

    } 

    sequence_size = keyword_size; 

} 

//the function call in main 
set_sequence("foo bar\n", 8); 

//there's supposed to be stuff done here with sequence that I haven't implemented yet 
free(sequence); //sequence is a global variable that I use the function to set 

FILE* dest = fopen("cipher", "w"); 

我包括在该文件位是因为它的存在,我得到的断言失败,但是当它注释掉的代码运行的精绝的原因(我没有这条线之后,因为我是试图找出问题)。

我通过valgrind的memcheck valgrind --tool=memcheck ../bin/cipher运行代码,看看问题是什么,但我无法理解它。这里的输出:

==10608== Memcheck, a memory error detector 
==10608== Copyright (C) 2002-2011, and GNU GPL'd, by Julian Seward et al. 
==10608== Using Valgrind-3.7.0 and LibVEX; rerun with -h for copyright info 
==10608== Command: ../bin/cipher -e testfile 
Program running in encrypt mode 
Open source file: success 
Allocate memory for raw_input: success 
Read source file: success 
Allocate memory for input: success 
input set-> freeing raw_input 
==10608== Invalid write of size 4 
==10608== at 0x80486DC: set_sequence (in /home/hugo/Programming/C++/Cipher 
==10608== by 0x8048A86: main (in /home/hugo/Programming/C++/Cipher/bin/cipher) 
==10608== Address 0x41f6688 is 0 bytes inside a block of size 8 alloc'd 
==10608== at 0x402BB7A: malloc (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so) 
==10608== by 0x804868C: set_sequence (in /home/hugo/Programming/C++/Cipher/bin/cipher 
==10608== by 0x8048A86: main (in /home/hugo/Programming/C++/Cipher/bin/cipher) 
==10608== 
==10608== HEAP SUMMARY: 
==10608==  in use at exit: 704 bytes in 2 blocks 
==10608== total heap usage: 6 allocs, 4 frees, 1,793 bytes allocated 
==10608== 
==10608== LEAK SUMMARY: 
==10608== definitely lost: 0 bytes in 0 blocks 
==10608== indirectly lost: 0 bytes in 0 blocks 
==10608==  possibly lost: 0 bytes in 0 blocks 
==10608== still reachable: 704 bytes in 2 blocks 
==10608==   suppressed: 0 bytes in 0 blocks 
==10608== Rerun with --leak-check=full to see details of leaked memory 
==10608== 
==10608== For counts of detected and suppressed errors, rerun with: -v 
==10608== ERROR SUMMARY: 2 errors from 1 contexts (suppressed: 0 from 0) 

回答

3

你没有给出序列的声明。它是一个char数组还是一个int数组?如果它是一个int数组,你的malloc是错误的,它需要分配keyword_size * sizeof(int)字节

+0

对不起,我花了一段时间才得到你的答案。我一直很忙。您的解决方案奏效我完全忘了'char'是1个字节,而'int'是4个字节,所以当然这是一种糟糕的做法,直到我尝试对'int'进行同样的操作时才会被忽视。 – Hugo