0

我需要帮助来检查我的代码是否正确。代码太大而无法完全包含，所以我将只粘贴受影响的部分。

char *tmp;
tmp=Decode_URL(tmp_data);
sprintf(Data,"%s",tmp);
tmp=Decode_URL(tmp_backup1);
sprintf(DataB[0],"%s",tmp);
tmp=Decode_URL(tmp_backup2);
sprintf(DataB[1],"%s",tmp);
tmp=Decode_URL(tmp_backup3);
sprintf(DataB[2],"%s",tmp);
tmp=Decode_URL(tmp_backup4);
sprintf(DataB[3],"%s",tmp);
tmp=Decode_URL(tmp_backup5);
sprintf(DataB[4],"%s",tmp);

该Decode_URL函数返回一个char *.

所以我的问题是，总是使用tmp接收char *函数返回的值是否正确？或者我应该创建更多char *tmpx，每次调用一个Decode_URL？

编辑更多信息：

char *Decode_URL(char *url){
      char *check;
      check=EnDeCrypt(some vars here);
      return check;
}

char *EnDeCrypt(const char *pszText, int iTextLen, const char *pszKey)
{
    char *cipher;
    int a, b, i=0, j=0, k;
    int ilen;
    int sbox[256];
    int key[256];

    ilen = strlen(pszKey);

    for (a=0; a < 256; a++)
    {
        key[a] = pszKey[a % ilen];
        sbox[a] = a;
    }

    for (a=0, b=0; a < 256; a++)
    {
        b = (b + sbox[a] + key[a]) % 256;
        swapints(sbox, a, b);
    }

    cipher = (char *)malloc(iTextLen);

    for (a=0; a < iTextLen; a++)
    {
        i = (i + 1) % 256;
        j = (j + sbox[i]) % 256;
        swapints(sbox, i, j);
        k = sbox[(sbox[i] + sbox[j]) % 256];
        cipher[a] = pszText[a] ^ k;
    }
    return cipher;
}

谢谢

4

5 回答 5

5

这当然取决于内存的来源，即Decode_URL()返回一个指向的指针。

如果它是一个静态数组，那么你的用法就很好。

如果它是动态分配的（由malloc()它的朋友或其任何朋友），那么你正在泄漏内存。

于 2013-04-26T12:06:06.600 回答
2

它是正确的。你也可以去

sprintf(DataB[4],"%s",Decode_URL(tmp_backup5))
于 2013-04-26T12:06:00.237 回答
0

我们真正需要知道的是 Decode_URL 做了什么来创建返回的 char 指针指向的内存，是 malloc 在这种情况下你需要释放它，它是静态的，在这种情况下你需要小心你如何使用函数和作为函数返回的数据将不是可重入的，也不是线程安全的。

于 2013-04-26T12:13:31.487 回答
0

这是正确的，因为 tmp 没有其他用途，但您可能需要考虑

一个宏：

#define SET_TEMP(BACKUP, TMP, DATA) \
        TMP=Decode_URL(BACKUP);     \
        sprintf(DATA,"%s",TMP)

并像这样使用它：

SET_TEMP(tmp_backup1, tmp, DataB[0]);
...

无论如何，你很高兴，宏只是一种更好的编码风格。

编辑：

看到你添加的代码后，你确实有内存问题。

这就是你的宏应该是这样的：

#define SET_TEMP(BACKUP, TMP, DATA) \
        TMP=Decode_URL(BACKUP);     \
        sprintf(DATA,"%s",TMP);     \
        free(TMP)

并像这样使用它：

SET_TEMP(tmp_backup1, tmp, DataB[0]);
...
于 2013-04-26T12:14:30.273 回答
0

只要你不需要对数据做更多的事情，当然，为什么不呢？

于 2013-04-26T12:05:51.290 回答