[Monetdb-developers] [Monetdb-checkins] MonetDB5/src/mal mal_authorize.mx, Nov2009, 1.82.4.3, 1.82.4.4

Martin Kersten Martin.Kersten at cwi.nl
Tue Nov 3 22:50:49 CET 2009


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.
> 




More information about the developers-list mailing list