authd: don't exit() on OPM duplicate scanner errors, just ignore it

I run into this code even with a single kind of scanner with a single
port, so I'm not even sure how it thinks it already exists.

Perhaps it's being parsed twice, or something similar.

c.f. issue #229
This commit is contained in:
Aaron Jones 2016-12-19 04:56:54 +00:00
parent df0c70dd1f
commit d1f8acb0da
No known key found for this signature in database
GPG key ID: EC6F86EE9CD840B5

View file

@ -812,7 +812,8 @@ create_opm_scanner(const char *key __unused, int parc __unused, const char **par
if(find_proxy_scanner(proxy->proto, proxy->port) != NULL) if(find_proxy_scanner(proxy->proto, proxy->port) != NULL)
{ {
warn_opers(L_CRIT, "OPM: got a duplicate scanner: %s (port %hu)", parv[0], proxy->port); warn_opers(L_CRIT, "OPM: got a duplicate scanner: %s (port %hu)", parv[0], proxy->port);
exit(EX_PROVIDER_ERROR); rb_free(proxy);
return;
} }
rb_dlinkAdd(proxy, &proxy->node, &proxy_scanners); rb_dlinkAdd(proxy, &proxy->node, &proxy_scanners);