Tickets :: [#8670] VFS->read() call parameters are not passed correctly

6.0.0-beta1

7/19/25

Summary	VFS->read() call parameters are not passed correctly
Queue	Ingo
Queue Version	1.2.2
Type	Bug
State	Resolved
Priority	1. Low
Owners	chuck (at) horde (dot) org
Requester	lstewart (at) room52 (dot) net
Created	10/29/2009 (5742 days ago)
Due
Updated	01/13/2010 (5666 days ago)
Assigned
Resolved	11/01/2009 (5739 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

01/13/2010 12:14:53 AM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in Git for this ticket: Change how VFS::read() is called to match the function definition ~~Bug: #8670~~ http://git.horde.org/diff.php/ingo/lib/Driver/Vfs.php?rt=horde-git&r1=9b22223f642b6ab65a71694ff34f709f06dfce9d&r2=012341c0238536a4f0bb774b124e82b5eaf666b5

11/01/2009 02:23:05 AM	Chuck Hagenbuch	Comment #4 Assigned to Chuck Hagenbuch State ⇒ Resolved	Reply to this comment
Committed, thanks.

11/01/2009 02:09:50 AM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in Git for this ticket: Change how VFS::read() is called to match the function definition ~~Bug: #8670~~ http://git.horde.org/diff.php/ingo/lib/Driver/Vfs.php?rt=horde-hatchery&r1=9b22223f642b6ab65a71694ff34f709f06dfce9d&r2=012341c0238536a4f0bb774b124e82b5eaf666b5

11/01/2009 02:09:45 AM	CVS Commit	Comment #2	Reply to this comment
Changes have been made in CVS for this ticket: merge from git: Fix VFS::read() calling convention ~~Bug: #8670~~ http://cvs.horde.org/diff.php/ingo/lib/Driver/vfs.php?rt=horde&r1=1.12.10.16&r2=1.12.10.17&ty=u

10/29/2009 06:16:48 PM	lstewart (at) room52 (dot) net	Comment #1 Priority ⇒ 1. Low New Attachment: ingo_fixread.diff Patch ⇒ Yes Milestone ⇒ Summary ⇒ VFS->read() call parameters are not passed correctly Type ⇒ Bug State ⇒ Unconfirmed Queue ⇒ Ingo	Reply to this comment
The patch makes the read() call pass parameters in the way the VFS code seems to expect them (gleaned from observing how the write code path works). I believe things may work without this patch but it seems to be more correct this way to me.