Bug 762255 (GLUSTER-523) - Hash type is not initialized in dht_layout_new
Summary: Hash type is not initialized in dht_layout_new
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-523
Product: GlusterFS
Classification: Community
Component: distribute
Version: mainline
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Anand Avati
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-01-07 13:39 UTC by Jeff Darcy
Modified: 2015-12-01 16:45 UTC (History)
4 users (show)

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

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

Description Jeff Darcy 2010-01-07 13:39:21 UTC 
CALLOC does zero the memory, and the enum value DHT_HASH_TYPE_DM does end up being zero as well, but relying on this correspondence is fragile at best.  Fixing this will probably involve hoisting the definition of dht_hashfn_type_t into hashfn.h so that dht-layout.c can see it.
Comment 1 Anand Avati 2010-01-23 16:07:08 UTC 
PATCH: http://patches.gluster.com/patch/2683 in master (dht: initialize layout type)
Note You need to log in before you can comment on or make changes to this bug.