系统API RFile::Seek的BUG

 
TInt RFile::Seek(TSeek aMode,TInt& aPos) const函数,当输入参数aPos超过文件范围时,函数不会返回错误值。
我在CFileSource类里面实现了这样一个函数
 
TInt CFileSource::DoSeek(TInt aPos)
 {
       TInt err = iRFile.Seek(ESeekStart, aPos);
 
       if(err == KErrNone)
       {
             iFilePos = aPos;
             iRdPos = iWrPos = iBufferSize;
             iReadPending = EFalse;
       }
 
       return err;
 }

以上实现看似没有问题,但当传入的参数aPos大于文件size时,iRFile.Seek(ESeekStart, aPos)返回KErrNone,导致外层逻辑出错。
 
我测试了一下,如果aPos为负数,iRFile.Seek()会返回KErrArgument,但aPos大于iFileSize时,返回KErrNone,这在SDK中并没有说明。
 
 
解决方法 1:函数外部保护
 
外层调用CFileSource::DoSeek(TInt aPos)时,检查参数,保证aPos不会越界。这种方法增加了外层的逻辑复杂性。
 
 
解决方法 2:函数内部保护
 
TInt CFileSource::DoSeek(TInt aPos)
 {
  if (aPos > iFileSize)
  {

   iEof = ETrue;

   return KErrEof;
  }
 
  TInt err = iRFile.Seek(ESeekStart, aPos);
  if(err == KErrNone)
  {
   iFilePos = aPos;
   iRdPos = iWrPos = iBufferSize;
   iReadPending = EFalse;

   iEof = (aPos == iFileSize) ? ETrue : EFalse;
  }
  return err;
 }

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值