解密中去除填充的算法有问题

[baoyongzhang]

这个算法有问题，这里应该是把后面填充的内容去掉吧，不足16填充几个几，这里忽略 0x0a，0A对应ascii码里的 \n ，这样的话，如果正好差 10 位不足 16，那就会填充 10 个 10 ，10 就是 0x0A，解密之后的原文后面会出现 10个 \n。

[BruceWind]

你的解密的秘文发我测试一下，这里 != 0x0a 是故意加的，有人解密的的原文里有换行，导致换行后的内容丢了。

[baoyongzhang]

我用 “111111” 6个原文测试，密文是 “yiCLSX8FMSs16AnRdrsscA==”。
这是密文，只要是原文长度除以16余10的情况都会出现，比如原文是6个1，或者22个1，

[baoyongzhang]

忘了，我这个密钥改了，你应该解密不了，你直接用 6个1加密在解密就行

[BruceWind]

private final String str = "111111";

    //下面的密文对应的原文：123abcABC&*(@#@#@)+_/中文测试
    String code = AESEncrypt.encode(this, str);

    Log.d("code", code + "");
    Log.d("code", AESEncrypt.decode(this,code) + "");

---

log :

07-19 11:36:52.364 5799-5799/com.androidyuan.aesjniencrypt D/code: rfTzn9WjsDFbK262m0k4xg==
07-19 11:36:52.365 5799-5799/com.androidyuan.aesjniencrypt D/code: 111111

---

我测试 ：看上去没有问题啊 ？

[BruceWind]

“这里应该是把后面填充的内容去掉吧，不足16填充几个几”
第一句说对了，第二句不对，没有填充，你看返回的Jstring真实的值，后面的字符被忽略了。

使用这个函数的位置：

    //去除结尾垃圾字符串 begin
    int index = findPaddingIndex(out);
    if(index==NULL)
    {
        return (char*)out;
    }
    if(index < strlen(out)){//  if (index>strlen)  will crash.
        memset(out+index, '\0', strlen(out)-index);
    }
    //去除结尾垃圾字符串 end

[baoyongzhang]

log里打印看不到\n。

断点可以看到，或者你打印的时候 + "end" 后面加字符串，打印出来可以看到好几个换行。

[BruceWind]

好的。我再看下 。。。

[BruceWind]

测试了，确实如此，这个问题，需要后面我来改下，我之前故意跳过了"\n"字符串，你急着使用到商业项目中的话,可以先把那个if里的东西删掉，但是，你的原文中一定不能有'\n' ，否则'\n'之后的东西就会丢失。

[BruceWind]

if (HEX[k]==c && HEX[k]!= 0x0a)

to

  if (HEX[k]==c)

[baoyongzhang]

好的，我也尝试解决一下

[BruceWind]

你弄好了，可以推一个pull request给我。

[BruceWind]

我大概知道怎么改了：

这个方法 findPaddingIndex(uint8_t * str)

for循环应该从后往前查，查到第一个有效字符的位置，停止。

    for (i = strlen(str); i >=0 ; i--) 

[baoyongzhang]

没办法确定什么是有效字符呀，就拿 \n 来说，填充的10个10的时候就是填充了 10个\n，解密之后应该去掉这10个10，但是你没办法知道这个\n是加密的时候填充的还是原文就存在的。
我的想法是，既然填充方法是不足16就填充几个几，可以直接取最后一个值，如果是10，就减去10，是多少减去多少，这样就加密填充的时候方法对应上的。

int findPaddingIndex(uint8_t * str)
{
    int len = strlen(str);
    return len - str[len - 1];
}

[BruceWind]

是的。

你可以试试 ，\n在开头 在中间 在结尾 ，各种情况，解密出来 有没有丢\n。

[baoyongzhang]

现在这种遍历的写法，必须有一个前提，那就是原文中不能存在 1-16 的字符，如果有就会被忽略掉，一般情况下确实不会出现，\n是10，除了\n常见之外，其他的很难碰到了。