Description of problem: gf_memdup() has only a single user, namely the stripe translator. Does it warrant being in libglusterfs for just a single user? It has an unused parameter. The stripe xlator updates its idea of how long the xattrs are even if the memory allocation failed. Version-Release number of selected component (if applicable): all How reproducible: code examination Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
REVIEW: http://review.gluster.org/5716 (stripe: remove unused param, handle mem alloc failure) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle)
REVIEW: http://review.gluster.org/5716 (stripe: remove unused param, handle mem alloc failure) posted (#2) for review on master by Kaleb KEITHLEY (kkeithle)
REVIEW: http://review.gluster.org/5716 (stripe: remove unused param, handle mem alloc failure) posted (#3) for review on master by Kaleb KEITHLEY (kkeithle)
COMMIT: http://review.gluster.org/5716 committed in master by Anand Avati (avati) ------ commit b880b6b2908ad4e4afc8e26613bd0db8f0b28750 Author: Kaleb S. KEITHLEY <kkeithle> Date: Wed Aug 28 22:43:41 2013 +0530 stripe: remove unused param, handle mem alloc failure Change-Id: I9c27b1edab111031ca8eea9cc49480ea01e39089 BUG: 1002207 Signed-off-by: Kaleb S. KEITHLEY <kkeithle> Reviewed-on: http://review.gluster.org/5716 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Anand Avati <avati>