andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

libglusterfs: Fix volume_options_t struct

Browse Source 
The volume_options_t struct was modified and a new member was introduced
in the middle of the struct. This caused GD2 to crash when it tried to
read the volume options. The new member has been moved to the end of the
struct to correct this.

And a note has been added to notify developers on how to modify this
struct, and the xlator_api_t struct.

Updates: gluster/glusterfs#302

Change-Id: I2e9899ec10516be29c7e9d574da53be8ec17a99e
Signed-off-by: Kaushal M <kaushal@redhat.com>
This commit is contained in:
Kaushal M 2018-03-01 17:34:05 +05:30
parent 7e7fd3595e
commit fc35d400cb
2 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
libglusterfs/src/options.h
View File

@ -82,6 +82,10 @@ typedef enum {
/* Each translator should define this structure */
/* XXX: This structure is in use by GD2, and SHOULD NOT be modified.
* If there is a need to add new members, add them to the end of the structure.
* If the struct must be modified, GD2 MUST be updated as well
*/
typedef struct volume_options {
char *key[ZR_VOLUME_MAX_NUM_KEY];
/* different key, same meaning */
@ -98,7 +102,6 @@ typedef struct volume_options {
* happen
*/
opt_validate_type_t validate;
opt_level_t level;
/* The op-version at which this option was introduced.
* This is an array to support options that get backported to supported
@ -131,6 +134,9 @@ typedef struct volume_options {
*/
char *setkey;
/* The level at which the option is classified
*/
opt_level_t level;
} volume_option_t;

11
libglusterfs/src/xlator.h
View File

@ -1052,6 +1052,10 @@ typedef struct {
/* This would be the only structure which needs to be exported by
the translators. For the backward compatibility, in 4.x series
even the old exported fields will be supported */
/* XXX: This struct is in use by GD2, and hence SHOULD NOT be modified.
* If the struct must be modified, see instructions at the comment with
* GD2MARKER below.
*/
typedef struct {
/* op_version: will be used by volume generation logic to figure
out whether to insert it in graph or no, based on cluster's
@ -1075,6 +1079,13 @@ typedef struct {
volume file, then that should be defined here. optional. */
volume_option_t *options;
/* XXX: GD2MARKER
* If a new member that needs to be visible to GD2 is introduced,
* add it above this comment.
* Any other new members need to be added below this comment, or at the
* end of the struct
*/
/* init(): mandatory method, will be called during the
graph initialization */
int32_t (*init) (xlator_t *this);