Bug 762417 (GLUSTER-685) - writing to a file fails complaining of "Transport endpoint not being connected"
Summary: writing to a file fails complaining of "Transport endpoint not being connected"
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-685
Product: GlusterFS
Classification: Community
Component: quota
Version: mainline
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Raghavendra G
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-02-25 10:36 UTC by Raghavendra G
Modified: 2015-12-01 16:45 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Raghavendra G 2010-02-25 10:36:47 UTC 
quota in its write call does,
1. gets stat of the file.
2. continues with write if quota is not exceeded.

between 1 and 2, the arguments are stored in a structure stored in frame->local. While doing so, the pointer to vector is stored instead of copying the entire vector structure. This results in accessing invalid memory during 2, hence causing invalid iov_base and iov_len. Due to this transport may not transfer the buffer, since iov_len may be arbitrarily high.
Comment 1 Anand Avati 2010-03-02 13:22:00 UTC 
PATCH: http://patches.gluster.com/patch/2818 in master (features/quota: store the entire vector in local instead of just a pointer to it in writev.)
Comment 2 Anand Avati 2010-03-02 13:22:07 UTC 
PATCH: http://patches.gluster.com/patch/2855 in master (features/quota: Remember '/' loc_t to initiate xattr calls.)
Comment 3 Anand Avati 2010-03-02 13:22:11 UTC 
PATCH: http://patches.gluster.com/patch/2819 in release-2.0 (features/quota: store the entire vector in local instead of just a pointer to it in writev.)
Comment 4 Anand Avati 2010-03-02 13:22:19 UTC 
PATCH: http://patches.gluster.com/patch/2857 in release-3.0 (features/quota: Remember '/' loc_t to initiate xattr calls.)
Comment 5 Anand Avati 2010-03-11 04:25:38 UTC 
PATCH: http://patches.gluster.com/patch/2818 in release-3.0 (features/quota: store the entire vector in local instead of just a pointer to it in writev.)
Note You need to log in before you can comment on or make changes to this bug.