diff options
| author | Volker Lendecke <vl@samba.org> | 2013-09-02 11:37:57 +0000 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2013-10-15 23:46:11 +0200 |
| commit | c938a1065fed2a001da69b6a6d826ef31be75003 (patch) | |
| tree | 2c4d7ee16998fa96b61630b7731cdd3ad41dacc1 /source3 | |
| parent | 4d85f91a5b309fac76ed8b0ed2a18132c18b2659 (diff) | |
| download | samba-c938a1065fed2a001da69b6a6d826ef31be75003.tar.gz samba-c938a1065fed2a001da69b6a6d826ef31be75003.tar.xz samba-c938a1065fed2a001da69b6a6d826ef31be75003.zip | |
smbd: Factor out remove_stale_share_mode_entries
Will be used in the next commit
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/locking/locking.c | 16 | ||||
| -rw-r--r-- | source3/locking/proto.h | 1 | ||||
| -rw-r--r-- | source3/locking/share_mode_lock.c | 15 |
3 files changed, 18 insertions, 14 deletions
diff --git a/source3/locking/locking.c b/source3/locking/locking.c index 7ac04a45e3..b5d4f24f34 100644 --- a/source3/locking/locking.c +++ b/source3/locking/locking.c @@ -693,6 +693,22 @@ bool share_mode_stale_pid(struct share_mode_data *d, uint32_t idx) return true; } +void remove_stale_share_mode_entries(struct share_mode_data *d) +{ + uint32_t i; + + i = 0; + while (i < d->num_share_modes) { + if (d->share_modes[i].stale) { + struct share_mode_entry *m = d->share_modes; + m[i] = m[d->num_share_modes-1]; + d->num_share_modes -= 1; + } else { + i += 1; + } + } +} + bool set_share_mode(struct share_mode_lock *lck, files_struct *fsp, uid_t uid, uint64_t mid, uint16 op_type) { diff --git a/source3/locking/proto.h b/source3/locking/proto.h index f6ae462bac..2c9654ce28 100644 --- a/source3/locking/proto.h +++ b/source3/locking/proto.h @@ -176,6 +176,7 @@ bool is_valid_share_mode_entry(const struct share_mode_entry *e); bool share_mode_stale_pid(struct share_mode_data *d, uint32_t idx); bool set_share_mode(struct share_mode_lock *lck, files_struct *fsp, uid_t uid, uint64_t mid, uint16 op_type); +void remove_stale_share_mode_entries(struct share_mode_data *d); bool del_share_mode(struct share_mode_lock *lck, files_struct *fsp); bool mark_share_mode_disconnected(struct share_mode_lock *lck, struct files_struct *fsp); diff --git a/source3/locking/share_mode_lock.c b/source3/locking/share_mode_lock.c index 342f9108d6..5d7a08ca4b 100644 --- a/source3/locking/share_mode_lock.c +++ b/source3/locking/share_mode_lock.c @@ -171,26 +171,13 @@ static TDB_DATA unparse_share_modes(struct share_mode_data *d) { DATA_BLOB blob; enum ndr_err_code ndr_err; - uint32_t i; if (DEBUGLEVEL >= 10) { DEBUG(10, ("unparse_share_modes:\n")); NDR_PRINT_DEBUG(share_mode_data, d); } - i = 0; - while (i < d->num_share_modes) { - if (d->share_modes[i].stale) { - /* - * Remove the stale entries before storing - */ - struct share_mode_entry *m = d->share_modes; - m[i] = m[d->num_share_modes-1]; - d->num_share_modes -= 1; - } else { - i += 1; - } - } + remove_stale_share_mode_entries(d); if (d->num_share_modes == 0) { DEBUG(10, ("No used share mode found\n")); |