Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
L
liblongpath-rsync
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
liblongpath
liblongpath-rsync
Commits
6969ebcf
Commit
6969ebcf
authored
Aug 20, 2003
by
Wayne Davison
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Call setgroups(1, &gid) after setting the gid (rather than calling
setgroups(0, NULL) before).
parent
1dbb94ca
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
9 additions
and
10 deletions
+9
-10
clientserver.c
clientserver.c
+9
-10
No files found.
clientserver.c
View file @
6969ebcf
...
...
@@ -344,16 +344,6 @@ static int rsync_module(int f_in, int f_out, int i)
}
if
(
am_root
)
{
#ifdef HAVE_SETGROUPS
/* Get rid of any supplementary groups this process
* might have inheristed. */
if
(
setgroups
(
0
,
NULL
))
{
rsyserr
(
FERROR
,
errno
,
"setgroups failed"
);
io_printf
(
f_out
,
"@ERROR: setgroups failed
\n
"
);
return
-
1
;
}
#endif
/* XXXX: You could argue that if the daemon is started
* by a non-root user and they explicitly specify a
* gid, then we should try to change to that gid --
...
...
@@ -369,6 +359,15 @@ static int rsync_module(int f_in, int f_out, int i)
io_printf
(
f_out
,
"@ERROR: setgid failed
\n
"
);
return
-
1
;
}
#ifdef HAVE_SETGROUPS
/* Get rid of any supplementary groups this process
* might have inheristed. */
if
(
setgroups
(
1
,
&
gid
))
{
rsyserr
(
FERROR
,
errno
,
"setgroups failed"
);
io_printf
(
f_out
,
"@ERROR: setgroups failed
\n
"
);
return
-
1
;
}
#endif
if
(
setuid
(
uid
))
{
rsyserr
(
FERROR
,
errno
,
"setuid %d failed"
,
(
int
)
uid
);
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment