diff options
| author | Robert Newson <rnewson@apache.org> | 2011-01-12 10:46:07 +0000 |
|---|---|---|
| committer | Robert Newson <rnewson@apache.org> | 2011-01-12 10:46:07 +0000 |
| commit | 43a32a5e88b0e296ccf4bdb1b56a7b087fccf814 (patch) | |
| tree | cf3b8ad3a9ac69ea556398b79cb37bf0adf0c47d /src/couchdb | |
| parent | e27bbbb380dc37d3b7a792636c2e10dea65b56d8 (diff) | |
ensure write_streamed_attachment bails on negative LenLeft values
While Filipe has identified the fix for COUCHDB-1021, this patch will
ensure that no other bug will cause negative values to be passed to
this function, in turn leading to database inflation problems, etc.
git-svn-id: https://svn.apache.org/repos/asf/couchdb/branches/1.1.x@1058059 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'src/couchdb')
| -rw-r--r-- | src/couchdb/couch_db.erl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/couchdb/couch_db.erl b/src/couchdb/couch_db.erl index 7522a189..7475b123 100644 --- a/src/couchdb/couch_db.erl +++ b/src/couchdb/couch_db.erl @@ -920,7 +920,7 @@ with_stream(Fd, #att{md5=InMd5,type=Type,encoding=Enc}=Att, Fun) -> write_streamed_attachment(_Stream, _F, 0) -> ok; -write_streamed_attachment(Stream, F, LenLeft) -> +write_streamed_attachment(Stream, F, LenLeft) when LenLeft > 0 -> Bin = F(), ok = couch_stream:write(Stream, Bin), write_streamed_attachment(Stream, F, LenLeft - size(Bin)). |