diff options
| author | Yaniv Kaul <ykaul@redhat.com> | 2019-06-08 01:22:27 +0300 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2019-08-01 04:12:07 +0000 |
| commit | d68e426263708334945dff69b88218ed71a48033 (patch) | |
| tree | c9739c005f4531493593e07c60594b4ad6944794 /cli | |
| parent | d7e449b0da35a30af876aef1bea1fb096d0c06dc (diff) | |
Multiple files: get trivial stuff done before lock
Initialize a dictionary for example seems to be prefectly fine to be done
before taking a lock.
Change-Id: Ib29516c4efa8f0e2b526d512beab488fcd16d2e7
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
Diffstat (limited to 'cli')
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index 9757fe338f3..dcf714bd862 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -3181,13 +3181,13 @@ print_quota_list_from_quotad(call_frame_t *frame, dict_t *rsp_dict) LOCK(&local->lock); { - ret = dict_get_int32(gd_rsp_dict, "quota-list-success-count", - &success_count); + ret = dict_get_int32_sizen(gd_rsp_dict, "quota-list-success-count", + &success_count); if (ret) success_count = 0; - ret = dict_set_int32(gd_rsp_dict, "quota-list-success-count", - success_count + 1); + ret = dict_set_int32_sizen(gd_rsp_dict, "quota-list-success-count", + success_count + 1); } UNLOCK(&local->lock); if (ret) { @@ -3286,12 +3286,13 @@ cli_quotad_getlimit_cbk(struct rpc_req *req, struct iovec *iov, int count, LOCK(&local->lock); { - ret = dict_get_int32(local->dict, "quota-list-count", &list_count); + ret = dict_get_int32_sizen(local->dict, "quota-list-count", + &list_count); if (ret) list_count = 0; list_count++; - ret = dict_set_int32(local->dict, "quota-list-count", list_count); + ret = dict_set_int32_sizen(local->dict, "quota-list-count", list_count); } UNLOCK(&local->lock); |