[RFC 3/3] libbb: read_full: use read return instead size
Alexander Aring
alex.aring at gmail.com
Mon Mar 3 04:04:13 EST 2014
Hi Sascha,
On Mon, Mar 03, 2014 at 09:30:50AM +0100, Sascha Hauer wrote:
>
> So our read_file implementation doesn't work procfs like filesystems where
> all sizes are 0. Your patch doesn't change this though. In 2/3 you
> remove the if (!size) check and call the fs drivers read function with
> size 0. In this case the read function may return values, but never
> actually read something because the buffer size is 0.
>
exactly. The imporant question for me is "do you want to accept patches
to changes this behaviour, so we can read zero file size files". I mean
this would be a huge change in internal api of filesystem layer and I
hope we doesn't break anything else.
But your are fine with me to say "there exists a problem with read of
zero sized files?".
I will create some "better" patches for this, this RFC should only answer
the question if you apply fixes or not.
- Alex
More information about the barebox
mailing list