[Monetdb-developers] [Monetdb-pf-checkins] pathfinder/runtime xrpc_server.mx, , 1.82, 1.83

Ying Zhang Y.Zhang at cwi.nl
Wed Jul 23 14:29:40 CEST 2008


Hi Stefan,

Thanks for cleaning up the code.  However, it is referred in MIL, so I  
think
this change will break loading the module.  This BAT should be used by
xrpc_handle_file_request() for security check.  I should add code to  
do this,
so I'd suggest to undo this change.  What do you think?

Regards,

Jennie


On Jul 23, 2008, at 12:37 , Stefan Manegold wrote:

> Update of /cvsroot/monetdb/pathfinder/runtime
> In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv9875/ 
> pathfinder/runtime
>
> Modified Files:
> 	xrpc_server.mx
> Log Message:
>
> clean-up: removing unused global variable
>
>
> U xrpc_server.mx
> Index: xrpc_server.mx
> ===================================================================
> RCS file: /cvsroot/monetdb/pathfinder/runtime/xrpc_server.mx,v
> retrieving revision 1.82
> retrieving revision 1.83
> diff -u -d -r1.82 -r1.83
> --- xrpc_server.mx	17 Jul 2008 21:28:28 -0000	1.82
> +++ xrpc_server.mx	23 Jul 2008 10:37:49 -0000	1.83
> @@ -194,7 +194,7 @@
> static MT_Sema xrpc_commit_sema;
> static xrpc_commit_t *xrpc_commit_active = NULL;
>
> -BAT *xrpc_trusted = NULL, *xrpc_admin = NULL, *xrpc_user = NULL;
> +BAT *xrpc_trusted = NULL, *xrpc_admin = NULL;
> char *xrpc_hostname = NULL;
> int xrpc_port = 0;
>
> @@ -889,7 +889,6 @@
>     @:xrpc_bat(admin,str)@
>     xrpc_admin = xrpc_bat;
>     @:xrpc_bat(user,str)@
> -    xrpc_user = xrpc_bat;
>
>     CMDmy_hostname(&xrpc_hostname);
>     MT_init_sema(xrpc_commit_sema,0, "XRPC_COMMIT_SEMA");
>
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's  
> challenge
> Build the coolest Linux based applications with Moblin SDK & win  
> great prizes
> Grand prize is a trip for two to an Open Source event anywhere in  
> the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Monetdb-pf-checkins mailing list
> Monetdb-pf-checkins at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/monetdb-pf-checkins





More information about the developers-list mailing list