From xaa@xaa.iae.nl  Mon Sep 14 14:17:46 1998
Received: from esmeralda.xaa.iae.nl (esmeralda.xaa.iae.nl [194.151.75.9])
          by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id OAA23379
          for <FreeBSD-gnats-submit@freebsd.org>; Mon, 14 Sep 1998 14:17:43 -0700 (PDT)
          (envelope-from xaa@xaa.iae.nl)
Received: from ariel.xaa.iae.nl (ariel.xaa.iae.nl [194.151.75.10])
	by esmeralda.xaa.iae.nl (VMailer) with ESMTP
	id 2A832566; Mon, 14 Sep 1998 23:17:24 +0200 (MET DST)
Received: by ariel.xaa.iae.nl (VMailer, from userid 1002)
	id 9A1B64217; Mon, 14 Sep 1998 23:17:23 +0200 (CEST)
Message-Id: <19980914211723.9A1B64217@ariel.xaa.iae.nl>
Date: Mon, 14 Sep 1998 23:17:23 +0200 (CEST)
From: xaa@xaa.iae.nl
Reply-To: xaa@xaa.iae.nl
To: FreeBSD-gnats-submit@freebsd.org
Subject: chpass with -c CLASS to change class
X-Send-Pr-Version: 3.2

>Number:         7929
>Category:       bin
>Synopsis:       extra option for chclass
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Mon Sep 14 14:20:00 PDT 1998
>Closed-Date:    Wed Sep 16 08:31:19 PDT 1998
>Last-Modified:  Wed Sep 16 08:31:59 PDT 1998
>Originator:     Mark Huizer
>Release:        FreeBSD 3.0-CURRENT i386
>Organization:
>Environment:

	current / 2.2.7 systems

>Description:

	there is no easy way to change loginclasses from scripts, the
	way you can do it with chpass -s SHELL user

>How-To-Repeat:

	

>Fix:

This patch will add chpass -c CLASSNAME to the possibilities, sorry,
haven't changed usage() and manpage yet.

--- usr.bin/chpass/chpass.c.orig	Mon Sep 14 22:43:55 1998
+++ usr.bin/chpass/chpass.c	Mon Sep 14 23:08:47 1998
@@ -82,7 +82,7 @@
 	int argc;
 	char **argv;
 {
-	enum { NEWSH, LOADENTRY, EDITENTRY, NEWPW, NEWEXP } op;
+	enum { NEWSH, NEWCLASS, LOADENTRY, EDITENTRY, NEWPW, NEWEXP } op;
 	struct passwd *pw, lpw;
 	char *username = NULL;
 	int ch, pfd, tfd;
@@ -94,9 +94,9 @@
 
 	op = EDITENTRY;
 #ifdef YP
-	while ((ch = getopt(argc, argv, "a:p:s:e:d:h:oly")) != -1)
+	while ((ch = getopt(argc, argv, "a:p:c:s:e:d:h:oly")) != -1)
 #else
-	while ((ch = getopt(argc, argv, "a:p:s:e:")) != -1)
+	while ((ch = getopt(argc, argv, "a:p:c:s:e:")) != -1)
 #endif
 		switch(ch) {
 		case 'a':
@@ -115,6 +115,10 @@
 			op = NEWEXP;
 			arg = optarg;
 			break;
+		case 'c':
+			op = NEWCLASS;
+			arg = optarg;
+			break;
 #ifdef YP
 		case 'h':
 #ifdef PARANOID
@@ -160,7 +164,7 @@
 
 	uid = getuid();
 
-	if (op == EDITENTRY || op == NEWSH || op == NEWPW || op == NEWEXP)
+	if (op == EDITENTRY || op == NEWSH || op == NEWPW || op == NEWEXP || op == NEWCLASS)
 		switch(argc) {
 #ifdef YP
 		case 0:
@@ -199,7 +203,12 @@
 		if (p_expire(arg, pw, (ENTRY *)NULL))
 			pw_error((char *)NULL, 0, 1);
 	}
-
+	if (op == NEWCLASS) {
+		if (uid)	/* only root can change class */
+			baduser();
+		if (p_class(arg, pw, (ENTRY *)NULL))
+			pw_error((char *)NULL, 0, 1);
+	}
 	if (op == LOADENTRY) {
 		if (uid)
 			baduser();
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->closed 
State-Changed-By: des 
State-Changed-When: Wed Sep 16 08:31:19 PDT 1998 
State-Changed-Why:  
The right backend for this kind of things is pw(8), which already has 
the functionality you suggest. 
>Unformatted:
