[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: DanteServer has a Timing Error Error on Startup?



Looks good to me.   I wonder where the race was exactly, but your fix
makes sense, and I don't have a problem with living without knowing
where the race was.  :)

Thanks for the patch!  Its committed to our tree.

-Pat

Craig Milo Rogers wrote:
> 	I think I see the error.  In "dante/DanteServer.java", there's
> the following code:
> 
>     public void start () throws Exception {
>         node = this.getNode();
>         neighbors = node.getNeighborTable();
> 
>         node.register(new DanteProtocol());
>         node.threadStart(this);
>     }
> 
> 	That's the wrong form of Node.register() to use;  DanteServer
> needs to use the two-argument call, where the second call is the Principal.
> The DabteServer object is being manipulated by AppBooster.run(), which has
> already passed the Principal to DanteServer.  So this ought to work:
> 
>         node.register(new DanteProtocol(), getPrincipal());
> 
> and it fixes the problem.
> 
> 					Craig Milo Rogers




[ Janos ] [ OSKit ] [ Network Testbed ] [ Flick ] [ Fluke ]
Flux Research Group / Department of Computer Science / University of Utah