Bug 1663247

Summary: remove static memory allocations from code
Product: [Community] GlusterFS Reporter: Sanju <srakonde>
Component: glusterdAssignee: Sanju <srakonde>
Status: CLOSED WONTFIX QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: mainlineCC: amukherj, bugs
Target Milestone: ---Keywords: EasyFix, Tracking, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-01-31 10:45:05 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 Sanju 2019-01-03 14:41:37 UTC 
Description of problem:
In the entire code base, many structures are allocating the memory statically. Instead we can allocate memory dynamically.

One such structure is:
struct glusterd_brickinfo {
    char hostname[NAME_MAX];
    char path[VALID_GLUSTERD_PATHMAX];
    char real_path[VALID_GLUSTERD_PATHMAX];
    char device_path[VALID_GLUSTERD_PATHMAX];
    char mount_dir[VALID_GLUSTERD_PATHMAX];
    char brick_id[1024];   /*Client xlator name, AFR changelog name*/
    char fstype[NAME_MAX]; /* Brick file-system type */
    char mnt_opts[1024];   /* Brick mount options */
..

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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
Comment 1 Sanju 2020-01-31 10:45:05 UTC 
As we are not planning to work on this in near future, closing this as won't fix.