1

我想使用 AES 算法。

我在上面使用这个功能

void SubBytes(char *SArr[4][4]){
int r,c;
char xlook[2];char ylook[2]; char hexStr[2]; char *Pad;
int intxlook,intylook;
char *temp;
int subbyte;
Pad = (char *)malloc (sizeof(char));
temp = (char *)malloc (sizeof(char));

for (r = 0; r < 4; r++)
  for(c = 0; c < 4; c++){
        strcpy(temp,"");
        strcpy(xlook,"");strcpy(ylook,"");
        strcpy(temp,SArr[r][c]);
        xlook[0] = temp[0]; xlook[1] = '\0';
        ylook[0] = temp[1];ylook[1] = '\0';
        intxlook = string_to_dec(xlook);
        intylook = string_to_dec(ylook);

     subbyte = SBox(intxlook,intylook);
     itoa(subbyte,hexStr,16);

     if (strlen(hexStr) < 2){
        strcpy(Pad,"");
        Pad = Padding(0,1);
        strcat(Pad,hexStr);  
        strcpy(SArr[r][c],Pad);
     }
     else
        strcpy(SArr[r][c],hexStr);

  }

  //free(temp); free(Pad);
}     //end

但我得到错误

围绕变量 'hexStr' 的堆栈在 c 中已损坏

4

2 回答 2

2

您忘记了 null-terminate hexStr。

所以当你这样做的时候strcpy(SArr[r][c],hexStr);。它将尝试复制到SArr[r][c]fromhexStr直到它到达 a '\0'，它可能在任何地方。

于 2012-07-17T08:02:41.323 回答
2

temp有足够的空间容纳单个字符：

temp = (char *)malloc (sizeof(char));

这意味着这将超出缓冲区的末尾，并可能导致损坏：

strcpy(temp,SArr[r][c]);

由于这是 C++，请使用std::string而不是char*.

于 2012-07-17T08:03:59.083 回答