Fabian Groffen wrote:
On 03-11-2009 22:22:54 +0100, Martin Kersten 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.
Fabian Groffen wrote: 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.