Roland Illig wrote:
+ if (!initialized) { + uid = getuid (); + ngroups = getgroups (0, NULL); + if (ngroups != -1) {
/* allocate space for one more element * than will be filled by getgroups(). */
+ groups = g_new (gid_t, ngroups + 1); + ngroups = getgroups (ngroups, groups); + + /* getgroups() may or may not return the effective group ID, + * so we always include it at the end of the list. */ + if (ngroups >= 0) { + groups[ngroups++] = getegid(); + } + } + initialized = TRUE; + }
I felt the need for that comment because otherwise the "+ 1" could become misinterpreted. Together with the second comment, it should be ok.
Roland _______________________________________________ Mc-devel mailing list http://mail.gnome.org/mailman/listinfo/mc-devel