diff options
| author | Jim Jagielski <jim@apache.org> | 2013-01-08 20:49:50 +0000 |
|---|---|---|
| committer | Jim Jagielski <jim@apache.org> | 2013-01-08 20:49:50 +0000 |
| commit | 18eb5b7bbba4ab254bd92381a553fa583a85a98d (patch) | |
| tree | 82e2fbd180c9e016975ff28d679d9d0f20445278 /modules/dav | |
| parent | 2d854077e308522fc9ab07987ef69d15ebb53c14 (diff) | |
| download | httpd-18eb5b7bbba4ab254bd92381a553fa583a85a98d.tar.gz | |
Merge r1331110 from trunk:
Replace use of apr_file_write() with apr_file_write_full() to prevent
incomplete writes.
Add comments in some places where error handling/logging is missing.
PR: 53131.
Submitted by: Nicolas Viennot <apache viennot biz>, Stefan Fritsch
Submitted by: sf
Reviewed/backported by: jim
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1430514 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'modules/dav')
| -rw-r--r-- | modules/dav/fs/lock.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/dav/fs/lock.c b/modules/dav/fs/lock.c index 331d4e8314..a15b4b91c7 100644 --- a/modules/dav/fs/lock.c +++ b/modules/dav/fs/lock.c @@ -845,7 +845,7 @@ static dav_error * dav_fs_save_locknull_list(apr_pool_t *p, const char *dirpath, } amt = pbuf->cur_len; - if ((rv = apr_file_write(file, pbuf->buf, &amt)) != APR_SUCCESS + if ((rv = apr_file_write_full(file, pbuf->buf, amt, &amt)) != APR_SUCCESS || amt != pbuf->cur_len) { err = dav_new_error(p, HTTP_INTERNAL_SERVER_ERROR, 0, rv, apr_psprintf(p, |