Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 831677

Summary: Add dict_set_transient_str
Product: [Community] GlusterFS Reporter: Jeff Darcy <jdarcy>
Component: coreAssignee: Jeff Darcy <jdarcy>
Status: CLOSED DEFERRED QA Contact:
Severity: low Docs Contact:
Priority: unspecified    
Version: 3.3-betaCC: bugs, gluster-bugs
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-12-14 19:40:28 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Jeff Darcy 2012-06-13 14:44:44 UTC
While reviewing http://review.gluster.com/#change,3565 I noticed that we have nearly a hundred uses of dict_set_dynstr, and a majority of those use gf_strdup'ed values.  This requires callers to check two return values, from gf_strdup and dict_set_dynstr, complicating error recovery.  I'd like to add a dict_set_transient_str function that does the gf_strdup internally, simplifying callers which then only need to check one return value.

Comment 1 Niels de Vos 2014-11-27 14:53:42 UTC
The version that this bug has been reported against, does not get any updates from the Gluster Community anymore. Please verify if this report is still valid against a current (3.4, 3.5 or 3.6) release and update the version, or close this bug.

If there has been no update before 9 December 2014, this bug will get automatocally closed.