On 09.12.2013 15:35, Michal Privoznik wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=981729
This config tunable allows users to determine the maximum number of accepted but yet not authenticated users.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com> --- daemon/libvirtd-config.c | 1 + daemon/libvirtd-config.h | 1 + daemon/libvirtd.aug | 1 + daemon/libvirtd.c | 1 + daemon/libvirtd.conf | 3 +++ daemon/test_libvirtd.aug.in | 1 + src/locking/lock_daemon.c | 4 ++-- src/lxc/lxc_controller.c | 2 +- src/rpc/virnetserver.c | 37 +++++++++++++++++++++++++++++++++++-- src/rpc/virnetserver.h | 1 + 10 files changed, 47 insertions(+), 5 deletions(-)
D'oh! I've made some changes but didn't amend them. Consider this squashed in: diff --git a/src/rpc/virnetserver.c b/src/rpc/virnetserver.c index bbb91d4..62490a7 100644 --- a/src/rpc/virnetserver.c +++ b/src/rpc/virnetserver.c @@ -1164,7 +1164,7 @@ void virNetServerRun(virNetServerPtr srv) "nclients_unauth=%zu nclients_unauth_max=%zu", srv->nclients, srv->nclients_max, srv->nclients_unauth, srv->nclients_unauth_max); - if ((srv->nclients == srv->nclients_max - 1) && + if ((srv->nclients < srv->nclients_max) && (srv->nclients_unauth < srv->nclients_unauth_max)) { /* Now it makes sense to accept() a new client. */ VIR_DEBUG("Re-enabling services"); @@ -1285,7 +1285,7 @@ size_t virNetServerClientAuth(virNetServerPtr srv, srv->nclients, srv->nclients_max, srv->nclients_unauth, srv->nclients_unauth_max); if ((srv->nclients < srv->nclients_max) && - (srv->nclients_unauth == srv->nclients_unauth_max - 1)) { + (srv->nclients_unauth < srv->nclients_unauth_max)) { /* Now it makes sense to accept() a new client. */ VIR_DEBUG("Re-enabling services"); virNetServerUpdateServicesLocked(srv, true); Michal