sandeepkru / kosmosfs

Automatically exported from code.google.com/p/kosmosfs
Apache License 2.0
0 stars 0 forks source link

bug in WriteAppender.cc #3

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
bug in file 
kfs-0.5/src/cc/libkfsClient/WriteAppender.cc
http://code.google.com/p/kosmosfs/source/browse/trunk/src/cc/libkfsClient/WriteA
ppender.cc

void SpaceRelease()
{
    if (mSpaceAvailable <= 0) {
            StartAppend();
        return;
    }
        Reset(mSpaceReleaseOp);
        mSpaceReleaseOp.chunkId      = mAllocOp.chunkId;
//! bug: should mAllocOp.chunkVersion; NOT ,
        mSpaceReleaseOp.chunkVersion = mAllocOp.chunkVersion; //! , --> ;
    mSpaceReleaseOp.writeInfo    = mWriteIds;
    mSpaceReleaseOp.numBytes     = size_t(mSpaceAvailable);
    Enqueue(mSpaceReleaseOp);
}
chenbk

Original issue reported on code.google.com by chenb...@gmail.com on 30 Nov 2011 at 3:10

GoogleCodeExporter commented 8 years ago

Original comment by srirams...@gmail.com on 16 Dec 2011 at 6:51

GoogleCodeExporter commented 8 years ago
Thanks for the find.  I have checked in a fix.

Original comment by srirams...@gmail.com on 16 Dec 2011 at 6:51

GoogleCodeExporter commented 8 years ago
The change for this is done now.

Original comment by srirams...@gmail.com on 16 Dec 2011 at 6:53

GoogleCodeExporter commented 8 years ago
Line 862 seems also suffer this issue.

Original comment by robinlee...@gmail.com on 22 Dec 2011 at 9:07