home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #30 / NN_1992_30.iso / spool / gnu / utils / bug / 2214 < prev    next >
Encoding:
Text File  |  1992-12-12  |  8.5 KB  |  335 lines
  1. Newsgroups: gnu.utils.bug
  2. Path: sparky!uunet!cis.ohio-state.edu!mercury.cs.mun.ca!michael
  3. From: michael@mercury.cs.mun.ca (Mike Rendell)
  4. Subject: fileutils-3.4: chgrp problems on old bsd systems, +fix
  5. Message-ID: <9212111827.AA19890@mercury.cs.mun.ca>
  6. Sender: gnulists@ai.mit.edu
  7. Organization: GNUs Not Usenet
  8. Distribution: gnu
  9. Date: Fri, 11 Dec 1992 11:09:55 GMT
  10. Approved: bug-gnu-utils@prep.ai.mit.edu
  11. Lines: 322
  12.  
  13. Machine: vax
  14. OS: (old) 4.3bsd from Mt. Xinu, Ultrix 2.2, others
  15.  
  16. The GNU chgrp doesn't work on older versions of bsd because only root can
  17. use chown(2) (there is a comment to this effect about sequents in the README
  18. file).  I've added (ifdef'd) code to the GNU chgrp to deal with such systems
  19. when it is installed setuid root.  The code checks that the user is in the
  20. named group, makes sure the user owns all files before changing the group,
  21. and checks directory permissions before recursing into them.  The diffs for
  22. configure(.in) add a test to see if the system has such a broken chown(2)
  23. (the test is done in a readable/executable subdirectory, conftest.dir, so it
  24. will work even if root is running configure).  The diffs also add a
  25. GETGROUPS_T test to the configure scripts since the modified chgrp uses
  26. getgroups().  Note that the Makefile doesn't install chgrp as setuid
  27. root - this needs to be done by hand.
  28.  
  29.  
  30. *** src/chgrp.c.orig    Fri Jul 17 17:18:38 1992
  31. --- src/chgrp.c    Fri Dec 11 14:41:06 1992
  32. ***************
  33. *** 24,32 ****
  34.   #include <getopt.h>
  35.   #include "system.h"
  36.   
  37. ! #ifndef _POSIX_VERSION
  38.   struct group *getgrnam ();
  39. ! #endif
  40.   
  41.   #ifdef _POSIX_SOURCE
  42.   #define endgrent()
  43. --- 24,48 ----
  44.   #include <getopt.h>
  45.   #include "system.h"
  46.   
  47. ! #ifdef _POSIX_VERSION
  48. ! #include <limits.h>
  49. ! #if !defined(NGROUPS_MAX) || NGROUPS_MAX < 1
  50. ! #define NGROUPS_MAX sysconf (_SC_NGROUPS_MAX)
  51. ! #endif /* !NGROUPS_MAX */
  52. ! 
  53. ! #else /* not _POSIX_VERSION */
  54.   struct group *getgrnam ();
  55. ! uid_t getuid ();
  56. ! gid_t getgid ();
  57. ! #include <sys/param.h>
  58. ! #if !defined(NGROUPS_MAX) && defined(NGROUPS)
  59. ! #define NGROUPS_MAX NGROUPS
  60. ! #endif /* not NGROUPS_MAX and NGROUPS */
  61. ! #endif /* not _POSIX_VERSION */
  62. ! 
  63. ! #ifndef GETGROUPS_T
  64. ! # define GETGROUPS_T gid_t
  65. ! #endif /* !GETGROUPS_T */
  66.   
  67.   #ifdef _POSIX_SOURCE
  68.   #define endgrent()
  69. ***************
  70. *** 63,68 ****
  71. --- 79,90 ----
  72.   /* The name of the group to which ownership of the files is being given. */
  73.   char *groupname;
  74.   
  75. + #ifdef HAVE_BROKEN_CHOWN
  76. + /* Uid/gid of the user running the program. */
  77. + uid_t    my_uid;
  78. + gid_t    my_gid;
  79. + #endif /* HAVE_BROKEN_CHOWN */
  80. + 
  81.   struct option long_options[] =
  82.   {
  83.     {"recursive", 0, 0, 'R'},
  84. ***************
  85. *** 113,118 ****
  86. --- 135,166 ----
  87.   
  88.     parse_group (argv[optind++], &group);
  89.   
  90. + #ifdef HAVE_BROKEN_CHOWN
  91. +   my_uid = getuid ();
  92. +   my_gid = getgid ();
  93. +   /* Check that (non-root) user is in group GROUP */
  94. +   if (my_uid)
  95. +     {
  96. + # ifdef NGROUPS_MAX
  97. +       GETGROUPS_T *groups;
  98. +       int i, n;
  99. + 
  100. +       groups = (GETGROUPS_T *) xmalloc (NGROUPS_MAX * sizeof(GETGROUPS_T));
  101. +       if ((n = getgroups (NGROUPS_MAX, groups)) < 0)
  102. +     error (1, errno, "cannot get supplemental group list");
  103. +       for (i = 0; i < n; i++)
  104. +     if (groups[i] == group)
  105. +       break;
  106. +       if (i == n)
  107. +     error (1, 0, "you are not in group %s", groupname);
  108. +       free (groups);
  109. + # else /* not NGROUPS_MAX */
  110. +       if (my_gid != group)
  111. +     error (1, 0, "you are not in group %s", groupname);
  112. + # endif /* not NGROUPS_MAX */
  113. +     }
  114. + #endif /* HAVE_BROKEN_CHOWN */
  115. + 
  116.     for (; optind < argc; ++optind)
  117.       errors |= change_file_group (argv[optind], group);
  118.   
  119. ***************
  120. *** 165,178 ****
  121.   
  122.     if (group != file_stats.st_gid)
  123.       {
  124. !       if (verbose)
  125. !     describe_change (file, 1);
  126. !       if (chown (file, file_stats.st_uid, group))
  127.       {
  128.         if (force_silent == 0)
  129. !         error (0, errno, "%s", file);
  130.         errors = 1;
  131.       }
  132.       }
  133.     else if (verbose && changes_only == 0)
  134.       describe_change (file, 0);
  135. --- 213,237 ----
  136.   
  137.     if (group != file_stats.st_gid)
  138.       {
  139. ! #ifdef HAVE_BROKEN_CHOWN
  140. !       if (my_uid && file_stats.st_uid != my_uid)
  141.       {
  142.         if (force_silent == 0)
  143. !         error (0, 0, "you are not the owner of %s", file);
  144.         errors = 1;
  145.       }
  146. +       else
  147. + #endif /* HAVE_BROKEN_CHOWN */
  148. +     {
  149. +       if (verbose)
  150. +         describe_change (file, 1);
  151. +       if (chown (file, file_stats.st_uid, group))
  152. +         {
  153. +           if (force_silent == 0)
  154. +         error (0, errno, "%s", file);
  155. +           errors = 1;
  156. +         }
  157. +     }
  158.       }
  159.     else if (verbose && changes_only == 0)
  160.       describe_change (file, 0);
  161. ***************
  162. *** 199,204 ****
  163. --- 258,274 ----
  164.     unsigned filelength;        /* Length of each pathname to process. */
  165.     unsigned pathlength;        /* Bytes allocated for `path'. */
  166.     int errors = 0;
  167. + 
  168. + #ifdef HAVE_BROKEN_CHOWN
  169. +   /* If only X_OK were used, chgrp's errors could be used to list the
  170. +      contents of unreadable directories.  So use R_OK as well... */
  171. +   if (my_uid && access (dir, R_OK|X_OK))
  172. +     {
  173. +       if (force_silent == 0)
  174. +     error (0, errno, "%s", dir);
  175. +       return 1;
  176. +     }
  177. + #endif /* HAVE_BROKEN_CHOWN */
  178.   
  179.     errno = 0;
  180.     name_space = savedir (dir, statp->st_size);
  181.  
  182.  
  183. *** configure.orig    Thu Oct 29 18:14:52 1992
  184. --- configure    Fri Dec 11 13:52:57 1992
  185. ***************
  186. *** 525,530 ****
  187. --- 525,582 ----
  188.   fi
  189.   rm -f conftest*
  190.   
  191. + echo checking getgroups group id type
  192. + echo "#include <sys/types.h>
  193. + #define NGID 256
  194. + main()
  195. + {
  196. +   gid_t gidset[NGID];
  197. +   int i, n;
  198. +   union { gid_t gval; long lval; }  val;
  199. + 
  200. +   val.lval = -1;
  201. +   for (i = 0; i < NGID; i++)
  202. +     gidset[i] = val.gval;
  203. +   n = getgroups(sizeof(gidset) / (sizeof(int) > sizeof(gid_t)
  204. +                               ? sizeof(int) : sizeof(gid_t)) - 1, gidset);
  205. +   if (n > 0 && gidset[n] != val.gval)
  206. +     exit(0);
  207. +   exit(1);
  208. + }" > conftest.c
  209. + eval $compile
  210. + if test -s conftest && ./conftest 2>/dev/null ; then
  211. +     DEFS="$DEFS -DGETGROUPS_T=int"
  212. + fi
  213. + rm -f conftest*
  214. + 
  215. + echo checking for broken chown
  216. + echo "main()
  217. + {
  218. +   int fd;
  219. +   char *file = "\"conftest.data\"";
  220. + 
  221. +   unlink (file);
  222. +   if ((fd = creat (file, 0644)) < 0)
  223. +     exit (1);
  224. +   close (fd);
  225. +   if (!getuid ())
  226. +     {
  227. +       if (chown (file, 1, getgid ()) < 0)
  228. +     exit (1);
  229. +       setuid (1);
  230. +     }
  231. +   if (chown (file, getuid (), getgid ()) < 0)
  232. +     exit (0);
  233. +   exit (1);
  234. + }" > conftest.c
  235. + eval $compile
  236. + [ -d conftest.dir ] || mkdir conftest.dir
  237. + chmod 755 conftest.dir
  238. + if test -s conftest && (cd ./conftest.dir; ../conftest) 2>/dev/null ; then
  239. +     DEFS="$DEFS -DHAVE_BROKEN_CHOWN"
  240. + fi
  241. + rm -rf conftest*
  242. + 
  243.   echo checking integer size
  244.   cat <<EOF > conftest.c
  245.   main() { exit(!(sizeof(long) > sizeof(int))); }
  246. ***************
  247. *** 715,720 ****
  248. --- 767,775 ----
  249.       LIBS="$SAVELIBS" ALLOCA=alloca.o
  250.     fi
  251.   fi
  252. + 
  253. + echo checking for BSD
  254. + ( test -f /vmunix || test -f /sdmach || test -f /../../mach ) && DEFS="$DEFS -DBSD42=1"
  255.   
  256.   echo checking for st_blocks in struct stat
  257.   echo "#include <sys/types.h>
  258.  
  259.  
  260. *** configure.in.orig    Thu Oct 29 16:27:10 1992
  261. --- configure.in    Fri Dec 11 13:56:32 1992
  262. ***************
  263. *** 90,95 ****
  264. --- 90,144 ----
  265.   fi
  266.   
  267.   AC_UID_T
  268. + echo checking getgroups group id type
  269. + echo "#include <sys/types.h>
  270. + #define NGID 256
  271. + main()
  272. + {
  273. +   gid_t gidset[NGID];
  274. +   int i, n;
  275. +   union { gid_t gval; long lval; }  val;
  276. + 
  277. +   val.lval = -1;
  278. +   for (i = 0; i < NGID; i++)
  279. +     gidset[i] = val.gval;
  280. +   n = getgroups (sizeof(gidset) / (sizeof(int) > sizeof(gid_t)
  281. +                               ? sizeof(int) : sizeof(gid_t)) - 1, gidset);
  282. +   if (n > 0 && gidset[n] != val.gval)
  283. +     exit (0);
  284. +   exit (1);
  285. + }" > conftest.c
  286. + eval $compile
  287. + if test -s conftest && ./conftest 2>/dev/null ; then
  288. +     DEFS="$DEFS -DGETGROUPS_T=int"
  289. + fi
  290. + rm -f conftest*
  291. + echo checking for broken chown
  292. + echo "main()
  293. + {
  294. +   int fd;
  295. +   char *file = "\"conftest.data\"";
  296. + 
  297. +   unlink (file);
  298. +   if ((fd = creat (file, 0644)) < 0)
  299. +     exit (1);
  300. +   close (fd);
  301. +   if (!getuid ())
  302. +     {
  303. +       if (chown (file, 1, getgid ()) < 0)
  304. +     exit (1);
  305. +       setuid (1);
  306. +     }
  307. +   if (chown (file, getuid (), getgid ()) < 0)
  308. +       exit (0);
  309. +   exit (1);
  310. + }" > conftest.c
  311. + [ -d conftest.dir ] || mkdir conftest.dir
  312. + chmod 755 conftest.dir
  313. + if test -s conftest && (cd ./conftest.dir; ../conftest) 2>/dev/null ; then
  314. +     DEFS="$DEFS -DHAVE_BROKEN_CHOWN"
  315. + fi
  316. + rm -rf conftest*
  317.   AC_INT_16_BITS
  318.   AC_STDC_HEADERS
  319.   AC_UNISTD_H
  320. ***************
  321. *** 117,122 ****
  322. --- 166,173 ----
  323.   
  324.   AC_VPRINTF
  325.   AC_ALLOCA
  326. + echo checking for BSD
  327. + ( test -f /vmunix || test -f /sdmach || test -f /../../mach ) && AC_DEFINE(BSD4_2)
  328.   AC_ST_BLOCKS
  329.   AC_UTIME_NULL
  330.   AC_XENIX_DIR
  331.  
  332. --
  333. Michael Rendell - Dept. of Comp. Sci., Memorial University of Newfoundland
  334.  
  335.