以解决

houtinghua

谢谢各位

houtinghua

每人来帮忙啊

samlumengjun

key[i] = bs[keyLocation+1+i];
这句有问题.  nkeylen = bs[keyLocation]; keyLocation = llen*0.382;
如果文件长度很短的话,keyLocation+1+i的值会超过llen.

huangwei0413

回复 1# houtinghua


    函数里面filename没有初始化，看看函数返回后有没有用到这个filename。
还有很多的内存泄露。。。

houtinghua

key = bs[keyLocation+1+i];
这句有问题.  nkeylen = bs[keyLocation]; keyLocation = llen*0.382;
如果文 ...
samlumengjun 发表于 2010-04-23 11:09

    这个问题我在加密的时候处理了一下  长度很小的时候 密钥也很小

houtinghua

回复 4# huangwei0413


    因为没有用到 所以没有初始化

samlumengjun

这个问题我在加密的时候处理了一下  长度很小的时候 密钥也很小
houtinghua 发表于 2010-04-23 13:27

    你没明白我的意思, bs申请的长度是llen,而llen 就是文件长度,假设文件长度是10, 而nkeylen的值是bs[3]的asc值,这个值的很有可能超过10, 假设bs[3]是'a',对应的asc值就是97. 而
for( i = 0;i<nkeylen;i++ )
                key = bs[keyLocation+1+i];
这里keyLocation+1+i就超过10了,bs越界,所以segment fault.

houtinghua

你没明白我的意思, bs申请的长度是llen,而llen 就是文件长度,假设文件长度是10, 而nkeylen的值是 ...
samlumengjun 发表于 2010-04-23 13:42

    再次感谢   在加密的时候我确定源文件长度必须大于45   否则否则不作加密处理

我写的代码不多  可能有很多内存泄露的地方  请大家帮忙指正

hellioncu

先不说别的，乘0.382是有潜在问题的，在某些情况下结果可能会相差1。
应该用 llen * 382 / 1000，这样才能确保每次结果都一样

samlumengjun

再次感谢   在加密的时候我确定源文件长度必须大于45   否则否则不作加密处理

我写的代码不多 ...
houtinghua 发表于 2010-04-23 13:49

    哎,都和你说那么明白了,还是不明白......
    问题是出在keyLocation+1+i这个是否大于llen,如果大于的话就有可能出错.
    再分析一下什么时候才能保证不出错, 要keyLocation+1+i不大于llen也就是文件长度.  keyLocation=llen*0.382,也就是最多0.4个文件长度,而i最大可以取到nkeylen也就是一个asc码的极限255. 也就是 0.4llen+256 < llen这个公式, 你的文件长度最少要取到427,才能保证所有情况都不会越界.不知道这样你能不能看懂.