Fabian Groffen wrote: > On 03-11-2009 22:22:54 +0100, Martin Kersten wrote: >> Fabian Groffen wrote: >>> On 03-11-2009 20:16:25 +0000, Martin Kersten wrote: >>>> Update of /cvsroot/monetdb/MonetDB5/src/mal >>>> In directory 23jxhf1.ch3.sourceforge.com:/tmp/cvs-serv5258 >>>> >>>> Modified Files: >>>> Tag: Nov2009 >>>> mal_authorize.mx >>>> Log Message: >>>> Test the language argument. Possibly null check. Discovered by Coverity >>> I think this commit breaks the original logic since the scenario bat may >>> be empty for e.g. the superuser. >> there is/should always be a scenario. The default being 'mal' > > IIRC (and I actually checked it, so I DO recall correctly), the contract > is: if there is none, you have access to all. This is the default, and > for sure for the admin user (monetdb). AFAICT, you introduced a > behavioural change which IMHO should go to Current, not the Candidate. ok, look fro another solution satisfying Coverity complaints. >
------------------------------------------------------------------------------ Come build with us! The BlackBerry(R) Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9 - 12, 2009. Register now! http://p.sf.net/sfu/devconference _______________________________________________ Monetdb-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/monetdb-developers
