C -malloc():最高尺寸损坏

Question

我正在研究一个程序，该程序解密文件中的一些文本。首先，我创建的另一个源代码要求Shift和一些文本。源代码加密文本并将其写入文件中。

然后，我尝试使用以下代码解密指定的文件 - 另一个源代码。获取文件的名称。从文件名来获得移位。在文件中写入的每个字符都会被复制，并借助函数caesar_decrypt函数转移值，而我也拥有的凯撒加密。但是，给定字符必须具有32至127之间的ASCII值，才能转换为32和127之间的ASCII值的另一个特征。

我定义了我的Modulo函数，封装了剩余的算子-Modulo，具有负数的正数和模量。我还定义了带状函数，该功能与python中的条函数相同。除了空间，它还删除了所有文字。我也有getin((，是python的输入((。

我试图通过打印和使用GDB进行调试。没有有效的结果。我已经在Stackoverflow中研究了这个主题。找到一个条目。答案没有解决我的特定问题。

注意：我复制了整个程序，因为我认为您需要必要的详细信息。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>
#include <unistd.h>
#include <fcntl.h>
/*GETIN FUNCTION*/
char* getIn() 
{
    char *line = NULL, *tmp = NULL;
    size_t size = 0, index = 0;
    int ch = EOF;
    while (ch) 
    {
        ch = getc(stdin);
        /* Check if we need to stop. */
        if (ch == EOF || ch == 'n')
        ch = 0;
        /* Check if we need to expand. */
        if (size <= index) 
        {
            size += sizeof(char);
            tmp = realloc(line, size);
            if (!tmp) 
            {
                free(line);
                line = NULL;
                break;
            }
        line = tmp;
        }
    /* Actually store the thing. */
    line[index++] = ch;
    }
    return line;
}
/*FUNCTION THAT CONVERTS TO NUMERIC CHARACTER, PRECURSOR OF DIGITS OF SHIFT*/
char make_numeric(char ch)
{
    if(ch<91 && ch>63) return ch-16;
    if(ch<123 && ch>95) return ch-48;
    if(ch<58 && ch>47) return ch;
    return ch+16;
}
 /*STRIP FUNCTION*/
char* strip(char* str,int length)
{
    char *start=str,*end=&str[length-1];
    while(*start>0 && *start<33 || *start==127) ++start;
    if(!*start) return start;
    while(*end>=0 && *end<33 || *end==127) --end;
    ++end;
    *end=0;
    return start;
}
/*DECRYPTOR FUNCTION*/
char *caesar_decrypt(char *message_to_decrypt, void *shift)
{   
    int length=strlen(message_to_decrypt),i;
    char* decrypted_message;
    if(!message_to_decrypt) return NULL;
    if(!(decrypted_message=calloc(length+1,sizeof(char)))) return NULL;
    for(i=0;i<length;++i)
    {
    decrypted_message[i]=mod((message_to_decrypt[i]-33-*((int*)shift)),94)+33;
    }
return decrypted_message;
}   
/*THE MAIN PROGRAM*/
int main()
{
int shift,len_chshift,init,len_filename,i=0;
char *chshift,*line=NULL,*decrypted_line,*stripped_line,chinit,chlen_chshift;
char *filename=NULL,*newfilename=NULL;
FILE *oldfile=NULL;
FILE *newfile=NULL;
size_t lenline,len=0;
printf("-9");
/*FILENAME*/
printf("Enter the file name: ");
/*printf("-8");*/
filename=getIn();
/*printf("-7");*/
if(!access(filename,F_OK))
{
    len_filename=strlen(filename);
    /*printf("-6");*/
    i=len_filename;
    while(filename[i]!='.') --i;        
    chlen_chshift=filename[i-1];
    chlen_chshift=make_numeric(chlen_chshift);
    len_chshift=chlen_chshift-48;
    /*printf("-5");*/
    chshift=calloc(len_chshift+1,1);
    /*NEWFILENAME*/
    newfilename=calloc(i+1,1);
    /*printf("-4");*/
    strncpy(newfilename,filename,i);
    /*printf("-3");*/
    strcat(newfilename,"(decrypted).txt");
    /*printf("-2");*/
    chinit=make_numeric(filename[0]);
    init=chinit-48;
    /*SHIFT*/
    i=0;
    /*printf("-1");*/
    while(i!=len_chshift)
    {
        chshift[i]=make_numeric(filename[(i+1)*init]);
        ++i;
    }
    /*printf("0");*/
    shift=atoi(chshift);
    /*printf("1");*/
    if(!(oldfile=fopen(filename,"r")))
    {
        perror("Error");
        if(newfilename) free(newfilename);
        if(filename) free(filename);
        if(chshift) free(chshift);
        exit(1);
    }
    /*printf("2");*/
    if(!(newfile=fopen(newfilename,"+ab")))
    {
        perror("Error");
        if(newfilename) free(newfilename);
        if(filename) free(filename);
        if(chshift) free(chshift);
        fclose(oldfile);
        exit(1);
    }
    while ((lenline = getline(&line, &len, oldfile)) != -1)
    {
        stripped_line=strip(line,lenline);
        decrypted_line=caesar_decrypt(stripped_line, &shift);
        if(!decrypted_line)
        {
            printf("Could not allocate memoryn");
            if(newfilename) free(newfilename);
            if(filename) free(filename);
            if(chshift) free(chshift);
            exit(1);
        }
        fprintf(newfile,"%sn",decrypted_line);
        if(decrypted_line) free(decrypted_line);
        decrypted_line=NULL;
        free(line);
        line=NULL;
        stripped_line=NULL;
        decrypted_line=NULL;
    }
    free(line);
    line=NULL;
}
else
{
    printf("Cannot access the file.");
    if(filename) free(filename);
    exit(0);
}
if(newfilename) free(newfilename);
if(filename) free(filename);
if(chshift) free(chshift);
fclose(oldfile);
fclose(newfile);
return 0;
}

程序需要创建一个文件，该文件的上下文是另一个文件的解密克隆，该文件的名称是用户在开始时提供的。但是，它在第一个Fopen之前就投掷了：

malloc(): corrupted top size
Aborted (core dumped)

当我输入每个printf行时，它在getin((和printf(" - 7"(的行之间会出现相同的错误。当对printfs进行评论时，我观察到文件名持有书面值，而getin((返回。因此，我认为Getin((不是问题。

edit(1(：我添加了2个丢失的标头，但仍会引起相同的错误。

edit(2(：我删除了所有不必要的标题。

edit(3(：由于我被要求，我会尝试以一个示例来照亮：假设我有一个名为

的文件

2/b*A".txt

有两行代码

%52
abcd

我在程序中输入文件名。

1(。是" (，转换为数字 2 。我们知道Shift是两位数的数字。<<strong>。/p>

2(第一个字符是启动器，即 2 ，是我们需要重复跳跃的字符数。由于我们有两个数字的偏移数，因此我们将跳过两个字符。

3(现在，我们有字符B和A分别转换为2和1。我们的班次是21.第一个字符％(37(向后移动到 n (110(等，我们需要创建一个带有内容的新文件：

n~#
LMNO

我感谢任何帮助。先感谢您。另外，我提到的变量是特定间隔之间的任何任意字符。有关详细信息，请检查源代码，或者我建议您在本地工作，因为我提供了整个程序。

对于ASCII代码：https://www.asciatible.com/

如果您对凯撒的密码感兴趣：https://en.wikipedia.org/wiki/caesar_cipher

Answer 1

newfilename中的缓冲区溢出：

/* here you declare that the string pointed to by newfilename may contain `i`
   chars + zero-termination...*/ 
newfilename=calloc(i+1,1);
/* Here `i` is less than `strlen(filename)`, so only i characters from `filename`
   will be copied, that does NOT include the zero-termination
   (but it will be zero-terminated because the last byte in the buffer was 
   already cleared by `calloc`) */ 
strncpy(newfilename,filename,i);

/* Here `newfilename` is full, and `strcat` will write outside buffer!! Whatever
   happens after this line is undefined behaviour */   
strcat(newfilename,"(decrypted).txt");

因此，您必须为newfilename分配足够的空间：

newfilename = calloc (i + strlen("(decrypted).txt") + 1, 1);