Comment 33 for bug 1196932

Revision history for this message
Mike Barton (redbo) wrote : Re: Possibly DoS attack using object tombstones

Yeah, we listdir the hash directory twice.

We probably don't need to do the first listdir for PUTs, POSTs and DELETEs, since we could just write the new file then do a cleanup op. But then I don't think we can do the other patch to reject operations that have no effect.

Or we could re-use the results of the first listdir to clean up, but that could leave multiple .data files laying around for concurrent PUTs.

Really, I'm not that excited about putting a lot of work into it, since the second listdir should generally be serviced by the page cache.