- developers - lists.mariadb.org

Re: [Maria-developers] [Commits] c934b5e: MDEV-8842 add group support to pam_user_map module.
by Sergei Golubchik 07 Oct '15

07 Oct '15

Hi, Holyfoot! On Oct 06, holyfoot(a)askmonty.org wrote: > revision-id: c934b5e661b19b7934d1a9af40afc70b727c2273 (mariadb-10.1.7-64-gc934b5e) > parent(s): 90f2c822469a4f88f882ba8974e790f0fb0b2702 > committer: Alexey Botchkov > timestamp: 2015-10-06 14:02:29 +0500 > message: > > MDEV-8842 add group support to pam_user_map module. > Added to the pam_user_map module. Right, thanks. A couple of comments, see below. > diff --git a/plugin/auth_pam/mapper/pam_user_map.c b/plugin/auth_pam/mapper/pam_user_map.c > index e73ab6d..0787b1c 100644 > --- a/plugin/auth_pam/mapper/pam_user_map.c > +++ b/plugin/auth_pam/mapper/pam_user_map.c ... > + /* > + So below we get the buffer of the NGROUPS_MAX size > + which can take 256K on some systems. > + It supposed to be working differently - we'd get > + the 'ng' from the getgrouplist() and alloc the respective > + size, but the problem is a bug in getgrouplist(). > + The glibc 2.3.2 implementation of this function is broken: > + it overwrites memory when the actual number of groups is larger > + than *ngroups. > + */ Forget 2.3.2, use it as it was supposed to be used. Our oldest builder uses glibc-2.5 > + (void) getgrouplist(user, user_group_id, groups, &ng); > + for (i= 0; i < ng; i++) > + { > + struct group *g; > + if ((g= getgrgid(groups[i])) == NULL) > + return 0; > + if (g->gr_gid == group_id) > + return 1; Eh? Why don't you compare group_id with groups[i] directly ? Regards, Sergei

2 1

Re: [Maria-developers] [Commits] 7ea735b: MDEV-8431 Feedback plugin needs an option for http proxy.
by Sergei Golubchik 06 Oct '15

06 Oct '15

Hi, Holyfoot! On Oct 05, holyfoot(a)askmonty.org wrote: > revision-id: 7ea735baa2df4f4577cf951bc69ccf87fd91f436 (mariadb-10.1.7-40-g7ea735b) > parent(s): ba0b6685510f4a271da8d386ff75bfd83d3156c2 > committer: Alexey Botchkov > timestamp: 2015-10-05 14:46:12 +0500 > message: > > MDEV-8431 Feedback plugin needs an option for http proxy. > 'feedback_http_proxy' system variable added to specify the > proxy server as host:port. Not a dynamic one. That's a bit more complex that I thought it'll be, but ok. Still - did you test it? How? Regards, Sergei

2 2

Re: [Maria-developers] [Commits] ee656a4: MDEV-7846: postreview fix
by Sergei Golubchik 06 Oct '15

06 Oct '15

Hi, Sanja! I have no idea how this postreview fix relates to any of my review comments. Let me repeat them again: 1. Add comments 2. Add a test case with row items 3. In your test case - if you remove GROUP BY you get a crash again. This might be a different bug, though. On Aug 05, sanja(a)mariadb.com wrote: > revision-id: ee656a4aabb5ebbe973b7e4c99be83eb5dc8c0ce > parent(s): 9f8a458fb2d07298810bb5d9824ce728017bfb32 > committer: Oleksandr Byelkin > branch nick: server > timestamp: 2015-08-05 11:57:35 +0200 > message: > > MDEV-7846: postreview fix > > diff --git a/sql/item_cmpfunc.cc b/sql/item_cmpfunc.cc > index 29fff8e..e96d1bc 100644 > --- a/sql/item_cmpfunc.cc > +++ b/sql/item_cmpfunc.cc > @@ -1455,12 +1455,13 @@ bool Item_in_optimizer::fix_left(THD *thd, Item **ref) > next execution we need to copy args[1]->left_expr again. > */ > ref0= &(((Item_in_subselect *)args[1])->left_expr); > + args[0]= ((Item_in_subselect *)args[1])->left_expr; > } > if ((!(*ref0)->fixed && (*ref0)->fix_fields(thd, ref0)) || > (!cache && !(cache= Item_cache::get_cache(*ref0)))) > DBUG_RETURN(1); > if (args[0] != (*ref0)) > - current_thd->change_item_tree(args, (*ref0)); > + args[0]= (*ref0); > DBUG_PRINT("info", ("actual fix fields")); Regards, Sergei

2 1

[Maria-developers] Problem in compiling Maria-db
by indiavaibhav vaibhavindia 02 Oct '15

02 Oct '15

Hello everyone, My name is Vaibhav.I am new to programming for open source. I am facing problems in trying to compile Maria-Db.As instructed on the website, I cloned the maria db source and installed all the files required for compilation. However on using cmake,I am getting an error,screenshot of which is attached is with this mail. Can someone please help me out? Thanks, Vaibhav Choudhary

2 3

Re: [Maria-developers] crosscompiling in buildroot: cannot find -laio and other link error
by Sylvain Raybaud 02 Oct '15

02 Oct '15

Ouch. I'm really sorry, I don't know what happend with my mua and why all these mails were sent to the list. Only the last one counts. Sincere apologies :'( On 02/10/15 17:10, Sylvain Raybaud wrote: > Hi Sergei! > > Thanks for your help. > > On 01/10/15 16:16, Sergei Golubchik wrote: >> Why would you build host variant for ha_connect.so with the library >> paths for the target architecture? > I don't actually, I don't know why this happened. > >> How do you invoke cmake? > The Buildroot system takes care of most of the maging. But... > >> Did you check >> https://mariadb.com/kb/en/mariadb/cross-compiling-mariadb/ ? Regards, >> Sergei > Ooops. For some reason I completely missed this page. Building only the > host variant of import_executables target eases cross-compilation a lot > indeed... Appart from that I had figured most of this out, or buildroot > was taking care of it. I still have a remark: I don't think I can > hardcode the value of > > HAVE_IB_GCC_ATOMIC_BUILTINS > > as suggested in https://lists.launchpad.net/maria-discuss/msg02911.html > because if I want this package accepted in buildroot it must > -- Sylvain Raybaud www.green-communications.fr

1 0

[Maria-developers] MariaDB Developer Meeting in Amsterdam Oct 13-15
by Rasmus Johansson 02 Oct '15

02 Oct '15

Hi maria-developers, There will be a MariaDB Developer Meeting taking place in Amsterdam on October 13-15. It would be great to see many of you there. The main focus of the event is as the title implies the development aspect of MariaDB, but also aspects like documentation, infrastructure and other community related topics will have their place in the event. Also MaxScale developers will be present. If you are interested please sign up by joining and rsvping the event's meetup page here: http://www.meetup.com/MariaDB-Developers/events/224068714/ The session backlog and the schedule for the three days are found here: https://docs.google.com/a/skysql.com/spreadsheets/d/17J_Jloc8I33nsEIE1KCZct… You'll notice that the schedule is not fixed and the idea is the following: - Everyone attending the meeting can affect the agenda by adding sessions to the session backlog - Everyone attending can/should vote on the session of interest in the backlog by raising the count in the Votes column Please feel free to suggest new sessions and remember to vote on the ones you think are most interesting (no limitation on how many you vote for). Best regards, Rasmus -- Rasmus Johansson, VP Engineering MariaDB | t: +358 50 499 9589 | Skype: ratzpo

2 2

Re: [Maria-developers] [Commits] 09eb231: MDEV-8762: file_key_management plugin should be disabled by default when linked statically
by Sergei Golubchik 01 Oct '15

01 Oct '15

Hi, Jan! That's not a good idea. It'd mean that installing a plugin is not enough, it must be explicitly enabled. I don't see any reason why one should be forced to do it. I'll comment in the bugreport. On Oct 01, Jan Lindström wrote: > revision-id: 09eb23148830c53e594cfb88dcb31f1b9e42ae5e (mariadb-10.1.7-33-g09eb231) > parent(s): 3a7cee05d6bac27c3c2f51afb8986c1581649bbc > committer: Jan Lindström > timestamp: 2015-10-01 14:27:10 +0300 > message: > > MDEV-8762: file_key_management plugin should be disabled by default when linked statically > > --- > sql/sql_plugin.cc | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/sql/sql_plugin.cc b/sql/sql_plugin.cc > index 6d8268e..1d2e2d8 100644 > --- a/sql/sql_plugin.cc > +++ b/sql/sql_plugin.cc > @@ -211,6 +211,7 @@ static struct > > /* we disable few other plugins by default */ > ,{ "feedback", PLUGIN_OFF } > + ,{ "file_key_management", PLUGIN_OFF } > }; Regards, Sergei

2 2

[Maria-developers] MDEV-8831: review request
by Nirbhay Choubey 01 Oct '15

01 Oct '15

Hi Jan, Could you please review the attached patch? Thanks, Nirbhay

1 0

[Maria-developers] Groups in PAM plugin.
by Alexey Botchkov 28 Sep '15

28 Sep '15

Hi. I came up with a small program that checks if an Unix user belongs to a group 'wheel'. That can be attached to the 'plugin/auth_pam/mapper/pam_user_map.c' in no time if that's what we want. http://myoffice.izhnet.ru/~af/gr.cc An it must be LDAP-compliant AFAIK. Best regards. HF

1 0

[Maria-developers] New Programmer(Maria-DB)
by indiavaibhav vaibhavindia 28 Sep '15

28 Sep '15

Hello everyone, My name is Vaibhav Choudhary.I am in my second year undergraduation and am new to open source. I would like to contribute to your project.Can anyone please guide me as to how should i work on a patch after i have identified the problem on JIRA? I have a good knowledge of C++ and am studying MySQL.But I am unable to figure out as to how should I go about writing the code for the issues on JIRA. Thanks Vaibhav

2 4