메모리를 할당 해제하기위한 free 명령을 호출하면 프로그램이 중단됩니다. 왜?

Question

나는 vigenere 암호를 사용하여 메시지를 암호화하고 암호 해독하는 프로그램을 작성했습니다. 암호화 부분을 잘 작동, 문제는 해독 부분입니다 : char * * free() 내 프로그램 충돌을 사용하여 할당 취소 할 때. 할당을 해제하지 않으면 프로그램이 제대로 작동하지만 충돌이 발생하지 않습니다.

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

int main(void) { 
    int choice, check, error; 
    char ch; 
    printf("What do you want to do?\n" 
      "1...Decode a message that is saved in a ciphertext.txt file.\n" 
      "2...Encode a message and save it in a ciphertext.txt file.\n"); 
    do { 
     error = 0; 
     check = scanf("%d%c", &choice, &ch); 
     if (check != 2 || ch != '\n' || choice < 1 || choice > 2) { 
      printf("Error: Invalid input!\n"); 
      error = 1; 
     } 
     fflush(stdin); 
    } while (error); 
    if (choice == 2) { 
     FILE *fp = fopen("ciphertext.txt", "w+"); 
     char buffer[1000]; 
     char *p, *q, *r; 
     char *encryption_text; 
     char *encrypted; 
     char *decrypted; 
     printf("What are bad characters?\n"); 
     printf("-Everything except A-Z and a-z.\n"); 
     printf("Plaintext: "); 
     gets(buffer); 
     decrypted = (char *)malloc(strlen(buffer)); 
     strcpy(decrypted, buffer); 
     printf("Key: "); 
     gets(buffer); 
     encryption_text = (char *)malloc(strlen(buffer)); 
     strcpy(encryption_text, buffer); 
     encrypted = (char *)malloc(strlen(decrypted)); 
     strcpy(encrypted, decrypted); 
     putchar('\n'); 
     p = decrypted; 
     q = encryption_text; 
     r = encrypted; 
     if (strlen(q) > strlen(p)) { 
      printf("The key has to be shorter or equal length as the plaintext."); 
      return 0; 
     } 
     while (*p != '\0') { 
      if (!(*p >= 'A' && *p <= 'Z') && !(*p >= 'a' && *p <= 'z')) { 
       printf("Bad characters."); 
       return 0; 
      } 
      if (*q == '\0') { 
       q = encryption_text; 
      } else if (!(*q >= 'A' && *q <= 'Z') && !(*q >= 'a' && *q <= 'z')) { 
       printf("Bad characters."); 
       return 0; 
      } 
      if (*p >= 'a' && *p <= 'z') *p -= ' '; 
      if (*q >= 'a' && *q <= 'z') *q -= ' '; 
      *r = (*p - 'A' + *q - 'A') % 26 + 'A'; 
      p++; 
      q++; 
      r++; 
     } 
     printf("Ciphertext: "); 
     puts(encrypted); 
     fputs(encryption_text, fp); 
     fprintf(fp, ";"); 
     fputs(encrypted, fp); 
     free(decrypted); 
     free(encrypted); 
     free(encryption_text); 
     fclose(fp); 
    } else { 
     FILE *fp = fopen("ciphertext.txt", "r+"); 
     char buffer[1000]; 
     char *encryption_text; 
     char *encrypted; 
     char *decrypted; 
     char delimeter[2] = ";"; 
     char *token; 
     char *p, *q, *r; 
     fgets(buffer, 1000, fp); 
     encrypted = (char *)calloc(strlen(buffer) * sizeof(char), sizeof(char)); 
     encryption_text = (char *)calloc(strlen(buffer) * sizeof(char), sizeof(char)); 
     token = strtok(buffer, delimeter); 
     strcpy(encryption_text, token); 
     encrypted = strrchr(buffer, '\0'); 
     if (encrypted != NULL) { 
      strcpy(encrypted, encrypted + 1); 
     } 
     decrypted = (char *)calloc(strlen(buffer) * sizeof(char), sizeof(char)); 
     for (p = decrypted, q = encryption_text, r = encrypted; 
       *r != '\0'; p++, r++, q++) { 
      if (*q == '\0') { 
       q = encryption_text; 
      } 
      *p = (*r - 'A' - (*q - 'A') + 26) % 26 + 'A'; 
     } 
     printf("Plaintext: "); 
     puts(decrypted); 
     printf("Key: "); 
     puts(encryption_text); 
     printf("Ciphertext: "); 
     puts(encrypted); 
     if (decrypted) free(decrypted); 
     if (encrypted) free(encrypted); 
     if (encryption_text) free(encryption_text); 
     fclose(fp); 
    } 
    return 0; 
} 

나는 그것이 충돌 이유를 말해 줄 수 있기를 바랍니다 : 다음은 프로그램 코드입니다. 감사!

Answer 1

충돌 주된 이유는

encrypted = strrchr(buffer, '\0'); 

이있는 당신은 지역 변수 buffer로 표현 메모리에 encrypted 지점 어딘가하자, 아마 다음 줄입니다. 이것은 스택의 가장 가능성있는 메모리이지만 적어도이 메모리는 malloc/calloc을 통해 할당되지 않았습니다.

나중에 encrypted을 해제하면 free- 운영시 충돌이 발생할 가능성이 높습니다.

BTW : 설명에 언급 된대로 strcpy을 사용할 때 malloc/calloc 문구를 다시 말하십시오. '\0'에 대한 공간이 있어야합니다.

또한 strrchr(buffer, '\0')은 buffer에 들어있는 문자열의 끝을 가리키는 포인터를 반환합니다. 두 번째 문자열 종료 문자가 있다는 보장이 없기 때문에 '\0'을 사용하여 종료 직후 위치에서 메모리를 복사하는 것이 중요 할 수 있습니다.

encrypted = strrchr(buffer, '\0'); 
     if (encrypted != NULL) { 
      strcpy(encrypted, encrypted + 1); 
     }