2010-06-14 37 views
7

Tôi phân bổ mảng 2d và sử dụng bộ nhớ để điền vào bằng số không."Sử dụng giá trị chưa được khởi tạo" mặc dù memset

#include<stdio.h> 
#include<string.h> 
#include<stdlib.h> 

void main() { 
    int m=10; 
    int n =10; 
    int **array_2d; 
    array_2d = (int**) malloc(m*sizeof(int*)); 
    if(array_2d==NULL) { 
     printf("\n Could not malloc 2d array \n"); 
     exit(1); 
    } 
    for(int i=0;i<m;i++) { 
     ((array_2d)[i])=malloc(n*sizeof(int)); 
     memset(((array_2d)[i]),0,sizeof(n*sizeof(int))); 
    } 


    for(int i=0; i<10;i++){ 
     for(int j=0; j<10;j++){ 
      printf("(%i,%i)=",i,j); 
      fflush(stdout); 
      printf("%i ", array_2d[i][j]); 
     } 
     printf("\n"); 
    } 
} 

Sau đó tôi sử dụng valgrind [1] để kiểm tra lỗi bộ nhớ. Tôi nhận được lỗi sau: Conditional jump or move depends on uninitialised value(s) cho dòng 24 (printf("%i ", array_2d[i][j]);). Tôi luôn nghĩ rằng memset là hàm khởi tạo các mảng. Làm thế nào tôi có thể thoát khỏi lỗi này?

Cảm ơn!

Valgrind đầu ra:

==3485== Memcheck, a memory error detector 
==3485== Copyright (C) 2002-2009, and GNU GPL'd, by Julian Seward et al. 
==3485== Using Valgrind-3.5.0-Debian and LibVEX; rerun with -h for copyright info 
==3485== Command: ./a.out 
==3485== 
(0,0)=0 (0,1)===3485== Use of uninitialised value of size 4 
==3485== at 0x409E186: _itoa_word (_itoa.c:195) 
==3485== by 0x40A1AD1: vfprintf (vfprintf.c:1613) 
==3485== by 0x40A8FFF: printf (printf.c:35) 
==3485== by 0x8048724: main (playing_with_valgrind.c:39) 
==3485== 
==3485== 
==3485== ---- Attach to debugger ? --- [Return/N/n/Y/y/C/c] ---- 
==3485== Conditional jump or move depends on uninitialised value(s) 
==3485== at 0x409E18E: _itoa_word (_itoa.c:195) 
==3485== by 0x40A1AD1: vfprintf (vfprintf.c:1613) 
==3485== by 0x40A8FFF: printf (printf.c:35) 
==3485== by 0x8048724: main (playing_with_valgrind.c:39) 

[1] valgrind --tool=memcheck --leak-check=yes --show-reachable=yes --num-callers=20 --track-fds=yes --db-attach=yes ./a.out

[gcc-cmd] gcc -std=c99 -lm -Wall -g3 playing_with_valgrind.c

Trả lời

14

Tại dòng này:

/* sizeof(n*sizeof(int)) retuns a value of type size_t. 
    This means you are initializing only sizeof(size_t) of the array. */ 
memset(((array_2d)[i]),0,sizeof(n*sizeof(int))); 

Nó nên là:

memset(((array_2d)[i]),0, n*sizeof(int)); 
+0

Đôi mắt tốt thực sự! –

+0

Argh! Cảm ơn nhiều. Tôi sẽ chấp nhận câu trả lời của bạn càng sớm càng tốt. – Framester

+0

Được phát hiện ... rất khó để xem – Spudd86

4

Thay đổi tuyên bố:

memset(((array_2d)[i]),0,sizeof(n*sizeof(int))); 

tới:

memset(((array_2d)[i]),0,n*sizeof(int)); 

Bạn không nên làm một sizeof đó. Nó sẽ chỉ trả về kích thước của loại biến, mà không phải là những gì bạn muốn.

Các vấn đề liên quan