Bug 1676797 - server xlator doesn't handle dict unserialization failures correctly
Summary: server xlator doesn't handle dict unserialization failures correctly
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: protocol
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-02-13 09:28 UTC by Pranith Kumar K
Modified: 2019-02-18 02:41 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-02-18 02:41:49 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 22210 0 None Open Fix compilation for fops-sanity.c 2019-02-14 08:22:07 UTC
Gluster.org Gerrit 22211 0 None Merged protocol/server: Use SERVER_REQ_SET_ERROR correctly for dicts 2019-02-18 02:41:48 UTC

Description Pranith Kumar K 2019-02-13 09:28:47 UTC
Description of problem:
On code reading found that there exists dead-code which was handling dict unserialization failures on server-rpc-fops.c.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Worker Ant 2019-02-13 10:24:18 UTC
REVIEW: https://review.gluster.org/22210 (Fix compilation for fops-sanity.c) posted (#1) for review on master by Pranith Kumar Karampuri

Comment 2 Worker Ant 2019-02-13 10:26:24 UTC
REVIEW: https://review.gluster.org/22211 (protocol/server: Use SERVER_REQ_SET_ERROR correctly for dicts) posted (#1) for review on master by Pranith Kumar Karampuri

Comment 3 Worker Ant 2019-02-14 08:22:09 UTC
REVIEW: https://review.gluster.org/22210 (Fix compilation for fops-sanity.c) merged (#2) on master by Amar Tumballi

Comment 4 Worker Ant 2019-02-18 02:41:49 UTC
REVIEW: https://review.gluster.org/22211 (protocol/server: Use SERVER_REQ_SET_ERROR correctly for dicts) merged (#2) on master by Pranith Kumar Karampuri


Note You need to log in before you can comment on or make changes to this bug.