Bugs item #1844050, was opened at 2007-12-04 12:54
Message generated for change (Settings changed) made by nielsnes
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1844050&group_id=56967
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: SQL/Core
Group: SQL CVS Head
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Romulo Goncalves (romulog)
Assigned to: Niels Nes (nielsnes)
Summary: Set a new user password...
Initial Comment:
I created a new user:
CREATE USER "voc" WITH PASSWORD 'voc' NAME 'VOC_EXPLORER' SCHEMA "sys";
CREATE SCHEMA "voc" AUTHORIZATION "voc";
ALTER USER "voc" SET SCHEMA "voc";
I alter the password user:
alter user "voc" with password 'new';
And then I start a mclient:
/mclient -lsql -uvoc
Password:new
the mserver crashes:
>mserver5: /ufs/goncalve/MonetDB/MonetDB5/src/mal/mal_authorize.mx:328:
>AUTHcheckCredentials: Assertion `q != 9223372036854775807LL' failed.
Program received signal SIGABRT, Aborted.
[Switching to Thread 1090525504 (LWP 30871)]
0x00000039b4c30065 in raise () from /lib64/libc.so.6
(gdb)
Regards,
Romulo
----------------------------------------------------------------------
>Comment By: Niels Nes (nielsnes)
Date: 2008-06-06 10:44
Message:
Logged In: YES
user_id=43556
Originator: NO
The last problem with this bug was that in the test the second server
started to quickly after the first
wasn't completely stopped. An extra delay solved this. So I'll now close
this.
----------------------------------------------------------------------
Comment By: Romulo Goncalves (romulog)
Date: 2008-05-29 14:36
Message:
Logged In: YES
user_id=1498628
Originator: YES
The bug is working in the stable branch, but not in the current cvs
branch.
So, it will remain open...
Romulo
----------------------------------------------------------------------
Comment By: Romulo Goncalves (romulog)
Date: 2008-05-29 14:34
Message:
Logged In: YES
user_id=1498628
Originator: YES
There is a test at:
sql/src/test/BugTracker/Tests/set_a_new_user_password.SF-1844050.py
with additional files:
set_a_new_user_password.SF-1844050_create_user.sql
set_a_new_user_password.SF-1844050_drop_user.sql
set_a_new_user_password.SF-1844050_select.sql
Romulo
----------------------------------------------------------------------
Comment By: Niels Nes (nielsnes)
Date: 2008-02-16 21:25
Message:
Logged In: YES
user_id=43556
Originator: NO
We used BUNinplace(b, p, BUNhead(), v, FALSE). In these cases BUNinplace
can fall back to delete/insert, which means that the value to which BUNhead
points could change. Now we use local variables instead.
A test should still be added.
----------------------------------------------------------------------
Comment By: Romulo Goncalves (romulog)
Date: 2008-01-08 15:01
Message:
Logged In: YES
user_id=1498628
Originator: YES
Well, the problem is back if I restart the Mserver.
I will not commit the changes and I will leave this bug open.
----------------------------------------------------------------------
Comment By: Fabian (mr-meltdown)
Date: 2008-01-08 14:37
Message:
Logged In: YES
user_id=963970
Originator: NO
problems here are:
1) I couldn't reproduce
2) Niels and I got stuck last year after finding that BUNinplace simply
did the insert, but the insert finally got lost on a restart.
Debugging/fixing 2) is way beyond my expertise, time limitations and
aspirations.
----------------------------------------------------------------------
Comment By: Romulo Goncalves (romulog)
Date: 2008-01-08 14:09
Message:
Logged In: YES
user_id=1498628
Originator: YES
If I do the following patch on MonetDB5
[EMAIL PROTECTED] mal]$ cvs diff
cvs diff: Diffing .
Index: mal_authorize.mx
===================================================================
RCS file: /cvsroot/monetdb/MonetDB5/src/mal/mal_authorize.mx,v
retrieving revision 1.55
diff -u -r1.55 mal_authorize.mx
--- mal_authorize.mx 4 Dec 2007 14:31:38 -0000 1.55
+++ mal_authorize.mx 8 Jan 2008 14:07:20 -0000
@@ -627,7 +627,7 @@
p = BUNfnd(pass, &id);
assert (p != BUN_NONE);
passi = bat_iterator(pass);
- BUNinplace(pass, p, BUNhead(passi, p), hash, FALSE);
+ BUNinplace(pass, p, BUNhead(passi, p), hash, TRUE);
GDKfree(hash);
AUTHcommit();
cvs diff: Diffing Tests
cvs diff: Diffing Tests/performanceTests
[EMAIL PROTECTED] mal]$
This crash is gone.
I ask to the God father of the authorization model if he aggrees with this
change?
If yes I will check in, create and approve test, and close the bug.
Regards,
Romulo
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=482468&aid=1844050&group_id=56967
-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Monetdb-bugs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/monetdb-bugs
