On 08/30/2016 12:42 PM, Pavel Březina wrote:
On 08/25/2016 01:43 PM, Petr Cech wrote:
> - /* FIXME: get max_children from configuration file */
> - auth_ctx->max_children = 10;
> + ret = confdb_get_int(be_ctx->cdb, be_ctx->conf_path,
> + CONFDB_PROXY_MAX_CHILDREN, 10,
> + &max_children);
> + if (ret != EOK) {
> + DEBUG(SSSDBG_CRIT_FAILURE, "Unable to read confdb [%d]: %s\n",
> + ret, sss_strerror(ret));
> + goto done;
> + }
> + if (max_children < 1) {
> + DEBUG(SSSDBG_CRIT_FAILURE, "Option %s must be bigger then 1\n",
> + CONFDB_PROXY_MAX_CHILDREN);
> + goto done;
> + }
You need to either set ret here, or set max_children to some reasonable
value (10?).
Hello Pavel,
max_children is set on the next line as:
auth_ctx->max_children = max_children;
I use temporary variable max_children,
because there is issue with signed/unsigned
integer value.
10 was original value. I increase it to 50.
And I use constant OPT_MAX_CHILDREN_DEFAULT now.
Fixed patch is attached.
Regards
--
Petr^4 Čech