Bu arabellek neden düzenlenemez baytlara işaret ediyor?

Question

DÜZENLEME: repliers sayesinde ben fread değişti (... sizeof 1, tampon, ...), ama şimdi valgrind bu hatayı alıyorum:

==2409== Invalid read of size 4 
==2409== at 0x51AB8D0: fread (iofread.c:41) 
==2409== by 0x4007B6: main (recover2.c:31) 
==2409== Address 0x5502000 is not stack'd, malloc'd or (recently) free'd 
==2409== 
==2409== Use of uninitialised value of size 8 
==2409== at 0x51B8787: _IO_sgetn (genops.c:495) 
==2409== by 0x51AB93E: fread (iofread.c:42) 
==2409== by 0x4007B6: main (recover2.c:31) 
==2409== 
==2409== Invalid read of size 8 
==2409== at 0x51B8787: _IO_sgetn (genops.c:495) 
==2409== by 0x51AB93E: fread (iofread.c:42) 
==2409== by 0x4007B6: main (recover2.c:31) 
==2409== Address 0x40 is not stack'd, malloc'd or (recently) free'd 
==2409== 
==2409== 
==2409== Process terminating with default action of signal 11 (SIGSEGV) 
==2409== Access not within mapped region at address 0x40 
==2409== at 0x51B8787: _IO_sgetn (genops.c:495) 
==2409== by 0x51AB93E: fread (iofread.c:42) 
==2409== by 0x4007B6: main (recover2.c:31) 
==2409== If you believe this happened as a result of a stack 
==2409== overflow in your program's main thread (unlikely but 
==2409== possible), you can try to increase the size of the 
==2409== main thread stack using the --main-stacksize= flag. 
==2409== The main thread stack size used in this run was 8388608. 

---

Burada yeniyim Bu yüzden umarım bu mantıklıdır. Bir dosyadan veri almak ve jpeg dosyalarına kopyalamak için bu kodu yazıyorum. Kod, bir jpg dosyasını kendi başlığına göre bulup daha sonra dosyalara yazar. Kod: Ben bir segment hataya alıyorum

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

int main(int argc, char* argv[]) 
{ 

FILE* file = fopen("card.raw", "r"); 
if (file == NULL) 
{ 
    printf("Could not open file!\n"); 
    return 1; 
} 

char title[7]; 
int currentImage = 0; 
uint8_t buffer[512]; 
FILE* img; 
while (fread(buffer, sizeof(buffer), 512, file) == 1) 
{ 
printf("found data!\n"); 
if (buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff) 
{ 
    if (buffer[3] == 0xe0 || buffer[3] == 0xe1 || buffer[3] == 0xe2 || buffer[3] == 0xe3 || buffer[3] == 0xe4 || buffer[3] == 0xe5 || buffer[3] == 0xe6 || buffer[3] == 0xe7 || buffer[3] == 0xe8 || buffer[3] == 0xe9 || buffer[3] == 0xea || buffer[3] == 0xeb || buffer[3] == 0xec || buffer[3] == 0xed || buffer[3] == 0xee || buffer[3] == 0xef) 
    { 
     printf("Found new jpg!\n"); 
     sprintf(title, "%03d.jpg", currentImage); 
     img = fopen(title, "a"); 
     currentImage++; 
     printf("size of buffer to print is %lu\n", sizeof(buffer)); 
     fwrite(buffer, sizeof(buffer), 1, img); 
     } 
} 
else if (currentImage > 0) 
{ 
     fwrite(buffer, sizeof(buffer), 1, img); 


} 

} 
} 

bir jpeg bulur ve fwrite yapar ve sonra while döngüsüne döndüğünde.

valgrind hatadır:

==1866== Syscall param read(buf) points to unaddressable byte(s) 
==1866== at 0x5228810: __read_nocancel (syscall-template.S:81) 
==1866== by 0x51B63B8: _IO_file_xsgetn (fileops.c:1438) 
==1866== by 0x51AB93E: fread (iofread.c:42) 
==1866== by 0x4007C3: main (recover2.c:31) 
==1866== Address 0xfff001000 is not stack'd, malloc'd or (recently) free'd 
==1866== 
==1866== Jump to the invalid address stated on the next line 
==1866== at 0x0: ??? 
==1866== Address 0x0 is not stack'd, malloc'd or (recently) free'd 
==1866== 
==1866== 
==1866== Process terminating with default action of signal 11 (SIGSEGV) 
==1866== Bad permissions for mapped region at address 0x0 
==1866== at 0x0: ??? 

Bu yüzden hala çok yanlış gitti neden anlamada herhangi bir yardım mutluluk duyacağız, bellek ve vb öğrenmeye yeniyim. 512 blok okumak için fread soruyorsun bu

fread(buffer, sizeof(buffer), 512, file) 

yaparak

Answer 1

, sizeof(buffer) bayt uzunluğunda olmak her bloğu. Yani uint8_t buffer[512] olarak bildirilen bir diziye 512 * 512 = 262144 bayt okumaya çalışıyorsunuz. Bu kesinlikle uygun değil. sadece buffer diziye veri okumak istiyorsanız İsterseniz

, o

fread(buffer, sizeof *buffer, sizeof buffer/sizeof *buffer, file) 

bağlı ya

fread(buffer, sizeof buffer, 1, file) 

veya

fread(buffer, 1, sizeof buffer, file) 

gelecektir ya da okuma işleminizde "atomik" veri bloğunu düşündüğünüz şey.

---

da

sprintf(title, "%03d.jpg", currentImage); 

title sıfır terminatörünü karşılamak için en az 8 karakterden olmak zorunda olduğu anlamına gelir, en az 7 karakter uzunluğunda bir dizi (örneğin, 001.jpg) oluşturur. Bu çok küçük

char title[7]; 

Ancak, sizin title bildirildi. fread() arasında man page uyarınca

Answer 2

, imza, her size bayt uzunluğunda açıklama işlevi fread() veri nmemb elemanları okur

olan

size_t fread(void *ptr, size_t size, size_t nmemb, FILE * stream);

olduğu akışındantarafından işaret edildi 210, onları ptr tarafından verilen yere saklar.

Yani, kod Aksi

while (fread(buffer, sizeof(buffer[0]), 512, file) == 1) 

olmalı, okumak ve yanlış büyüklükte bloklar 512 byte her 512 sayısını depolamak isteyen biten ediyoruz ve tampon neden olur valgrind tarafından bildirildiği gibi taşma.