Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

FileRequest: make sure we're dealing with a file.

Browse Source 
Trying to Read from a directory results in an error code, but we also
missed that because an unsigned variable was used to store the result.

Fixes #10210.
This commit is contained in:
Adrien Destugues 2013-11-18 09:53:04 +01:00
parent 94fb954c53
commit ba3f67bfb4
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/kits/network/libnetapi/FileRequest.cpp
View File

@ -35,7 +35,7 @@ BFileRequest::_ProtocolLoop()
// FIXME error handling (file does not exists, etc.)
BFile file(fUrl.Path().String(), B_READ_ONLY);
if(file.InitCheck() != B_OK)
if(file.InitCheck() != B_OK || !file.IsFile())
return B_PROT_CONNECTION_FAILED;
// Send all notifications to listener, if any
@ -45,7 +45,7 @@ BFileRequest::_ProtocolLoop()
file.GetSize(&size);
fListener->DownloadProgress(this, size, size);
size_t chunkSize;
ssize_t chunkSize;
char chunk[4096];
while((chunkSize = file.Read(chunk, sizeof(chunk))) > 0)
fListener->DataReceived(this, chunk, chunkSize);