Patch for Ticket 6344

Greg Hudson ghudson at MIT.EDU
Mon Apr 29 13:23:41 EDT 2013


On 04/29/2013 02:13 AM, bwleake wrote:
> A first 
> draft of the patch is up at github.com/pwillred/krb5, under the "listen" 
> branch.

Thanks!  I took a brief look.  Can you take a look at:

    http://k5wiki.kerberos.org/wiki/Contributing_code

and try to get this more in accordance with our C style?  The style
checker script should help.

> I am somewhat concerned that running two kdc or kadmin daemons 
> concurrently could result in incorrect or undesired behavior.

This shouldn't be a problem.  We already have krb5kdc -w which forks off
multiple KDC processes.

> Additionally, I'm unsure if this change would need to go through the 
> project planning process for 1.12 or 1.13.

We don't need a project proposal for something this simple.  We will
need documentation of the new option in
doc/admin/admin_commands/krb5kdc.rst and kadmind.rst.

Ordinarily we would require automated tests for a new feature, but I'm
not sure if we can test this without making assumptions about the
builder's network configuration.



More information about the krbdev mailing list