diff options
author | Vijay Bellur <vbellur@redhat.com> | 2018-09-24 13:21:58 -0700 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-09-25 17:01:43 +0000 |
commit | 8f207feb260bc6112d5085859b43efc8378a4c5f (patch) | |
tree | 2b1ec11256742e8d564de469b5d6476311beb210 /xlators/features/marker/src/marker.c | |
parent | f1b2455b9d01bae8c4652fc3583a60ff7000410e (diff) |
features/marker: Avoid passing structures as parameters to functions
Fixes CIDs: 1388779,1390242,1390057
The patch does the following:
1. Changes the prototype of mq_inspect_file_xattr() and mq_xattr_state()
to have a parameter of struct iatt * instead of struct iatt.
2. Remove unneeded struct iatt parameter from
mq_inspect_directory_xattr()
Change-Id: I1f6eabda042f0fe56d8c9bca010302f5e518dbe4
updates: bz#789278
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/features/marker/src/marker.c')
-rw-r--r-- | xlators/features/marker/src/marker.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/features/marker/src/marker.c b/xlators/features/marker/src/marker.c index d3abd4b0b5b..29294e73fd4 100644 --- a/xlators/features/marker/src/marker.c +++ b/xlators/features/marker/src/marker.c @@ -2966,7 +2966,7 @@ unwind: gf_uuid_copy(local->loc.gfid, buf->ia_gfid); if (priv->feature_enabled & GF_QUOTA) { - mq_xattr_state(this, &local->loc, dict, *buf); + mq_xattr_state(this, &local->loc, dict, buf); } out: @@ -3118,7 +3118,7 @@ marker_readdirp_cbk(call_frame_t *frame, void *cookie, xlator_t *this, "failed for %s", uuid_utoa(loc.inode->gfid)); - mq_xattr_state(this, &loc, entry->dict, entry->d_stat); + mq_xattr_state(this, &loc, entry->dict, &entry->d_stat); loc_wipe(&loc); ret = marker_key_set_ver(this, entry->dict); |