From 1373a11f571567c57a8f0084db61b2d32a965f24 Mon Sep 17 00:00:00 2001 From: Nikos Mavrogiannopoulos Date: Fri, 16 Jun 2023 19:12:46 +0200 Subject: [PATCH] tests: added a test for groups defined over multiple AVPs This adds a test for the available multi-group options as well as documentation for the feature. This tests two options: * Separate group names in separate class attributes * Separate group names in separate class attributes with the OU= format Signed-off-by: Nikos Mavrogiannopoulos --- doc/README-radius.md | 7 +- tests/Makefile.am | 5 +- tests/data/raddb/users | 23 ++ tests/data/radius-multi-group.config | 305 +++++++++++++++++++++++++++ tests/radius-multi-group | 246 +++++++++++++++++++++ 5 files changed, 582 insertions(+), 4 deletions(-) create mode 100644 tests/data/radius-multi-group.config create mode 100755 tests/radius-multi-group diff --git a/doc/README-radius.md b/doc/README-radius.md index 1978f397..205a9f53 100644 --- a/doc/README-radius.md +++ b/doc/README-radius.md @@ -102,10 +102,13 @@ ATTRIBUTE Framed-IP-Netmask 9 ipaddr # a CIDR string) ATTRIBUTE Framed-Route 22 string -# Sets group name using format "OU=group1;group2" +# There are two ways to set a group, either in the +# format "OU=group1;group2" or by a single group name +# in the attribute. It is possible to specify multiple +# groups in separate class attributes. # Note that this works only when groupconfig is set to # true, and if the groups sent by the server are made known -# to ocserv, via the select-group variable. +# to ocserv, via the select-group config variable. ATTRIBUTE Class 25 string # sets DNS servers diff --git a/tests/Makefile.am b/tests/Makefile.am index 86983274..7427a940 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -45,7 +45,8 @@ EXTRA_DIST = certs/ca-key.pem certs/ca.pem ns.sh common.sh certs/server-cert.pem data/haproxy-proxyproto.cfg scripts/proxy-connectscript data/haproxy-proxyproto-v1.config \ data/haproxy-proxyproto-v1.cfg scripts/proxy-connectscript-v1 data/test-multiple-client-ip.config \ data/test-client-bypass-protocol.config asan.supp certs/ca.tmpl certs/server-cert.tmpl \ - certs/user-cert.tmpl data/test-camouflage.config data/test-camouflage-norealm.config + certs/user-cert.tmpl data/test-camouflage.config data/test-camouflage-norealm.config \ + data/radius-multi-group.config xfail_scripts = dist_check_SCRIPTS = ocpasswd-test @@ -66,7 +67,7 @@ dist_check_SCRIPTS += haproxy-connect test-iroute test-multi-cookie test-pass-sc test-camouflage-norealm if RADIUS_ENABLED -dist_check_SCRIPTS += radius-group radius-otp +dist_check_SCRIPTS += radius-group radius-multi-group radius-otp endif #other tests requiring nuttcp for traffic diff --git a/tests/data/raddb/users b/tests/data/raddb/users index 61ea4a8e..be191467 100644 --- a/tests/data/raddb/users +++ b/tests/data/raddb/users @@ -101,6 +101,29 @@ test-class Cleartext-Password := "test-class" Class = "unknown", Framed-MTU = 1500 +test-multi-class Cleartext-Password := "test-multi-class" + Service-Type = Framed-User, + Framed-Protocol = PPP, + Framed-IP-Address = 192.168.94.192, + Framed-IP-Netmask = 255.255.255.0, + Framed-Routing = Broadcast-Listen, + Class = "group1", + Class = "group2", + Class = "group3", + Class = "group4", + Class = "unknown", + Framed-MTU = 1500 + +test-multi-class-ou Cleartext-Password := "test-multi-class-ou" + Service-Type = Framed-User, + Framed-Protocol = PPP, + Framed-IP-Address = 192.168.94.192, + Framed-IP-Netmask = 255.255.255.0, + Framed-Routing = Broadcast-Listen, + Class = "OU=group1;group2", + Class = "OU=group3;group4", + Framed-MTU = 1500 + testtime Cleartext-Password := "test" Service-Type = Framed-User, Framed-Protocol = PPP, diff --git a/tests/data/radius-multi-group.config b/tests/data/radius-multi-group.config new file mode 100644 index 00000000..39e80c6f --- /dev/null +++ b/tests/data/radius-multi-group.config @@ -0,0 +1,305 @@ +# User authentication method. Could be set multiple times and in that case +# all should succeed. +# Options: certificate, pam. +#auth = "certificate" +#auth = "plain[/etc/ocserv/passwd]" +#auth = "pam" +auth = "radius[config=@SRCDIR@/data/radiusclient/radiusclient.conf,groupconfig=true,nas-identifier=ocserv1]" +#enable-auth = "certificate" + +# Whether to enable support for the occtl tool (i.e., either through D-BUS, +# or via a unix socket). +occtl-socket-file = @OCCTL_SOCKET@ +use-occtl = true + +select-group = group1 +select-group = group2 +select-group = group3 +select-group = group4 + +isolate-workers = @ISOLATE_WORKERS@ + +# socket file used for IPC with occtl. You only need to set that, +# if you use more than a single servers. +#occtl-socket-file = /var/run/occtl.socket + +# The plain option requires specifying a password file which contains +# entries of the following format. +# "username:groupname:encoded-password" +# One entry must be listed per line, and 'ocpasswd' can be used +# to generate password entries. +#auth = "plain[/etc/ocserv/ocpasswd]" + +acct = "radius[config=@SRCDIR@/data/radiusclient/radiusclient.conf]" + +# A banner to be displayed on clients +#banner = "Welcome" + +stats-report-time = 30 + +# Use listen-host to limit to specific IPs or to the IPs of a provided +# hostname. +#listen-host = @ADDRESS@ + +# Limit the number of clients. Unset or set to zero for unlimited. +#max-clients = 1024 +max-clients = 16 + +# Limit the number of client connections to one every X milliseconds +# (X is the provided value). Set to zero for no limit. +#rate-limit-ms = 100 + +# Limit the number of identical clients (i.e., users connecting +# multiple times). Unset or set to zero for unlimited. +max-same-clients = 2 + +# TCP and UDP port number +tcp-port = @PORT@ +udp-port = @PORT@ + +# Keepalive in seconds +keepalive = 32400 + +# Dead peer detection in seconds. +dpd = 240 + +# Dead peer detection for mobile clients. The needs to +# be much higher to prevent such clients being awaken too +# often by the DPD messages, and save battery. +# (clients that send the X-AnyConnect-Identifier-DeviceType) +mobile-dpd = 1800 + +# MTU discovery (DPD must be enabled) +try-mtu-discovery = false + +# The key and the certificates of the server +# The key may be a file, or any URL supported by GnuTLS (e.g., +# tpmkey:uuid=xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxx;storage=user +# or pkcs11:object=my-vpn-key;object-type=private) +# +# There may be multiple certificate and key pairs and each key +# should correspond to the preceding certificate. +server-cert = @SRCDIR@/certs/server-cert.pem +server-key = @SRCDIR@/certs/server-key.pem + +# Diffie-Hellman parameters. Only needed if you require support +# for the DHE ciphersuites (by default this server supports ECDHE). +# Can be generated using: +# certtool --generate-dh-params --outfile /path/to/dh.pem +#dh-params = /path/to/dh.pem + +# If you have a certificate from a CA that provides an OCSP +# service you may provide a fresh OCSP status response within +# the TLS handshake. That will prevent the client from connecting +# independently on the OCSP server. +# You can update this response periodically using: +# ocsptool --ask --load-cert=your_cert --load-issuer=your_ca --outfile response +# Make sure that you replace the following file in an atomic way. +#ocsp-response = /path/to/ocsp.der + +# In case PKCS #11 or TPM keys are used the PINs should be available +# in files. The srk-pin-file is applicable to TPM keys only, and is the +# storage root key. +#pin-file = /path/to/pin.txt +#srk-pin-file = /path/to/srkpin.txt + +# The Certificate Authority that will be used to verify +# client certificates (public keys) if certificate authentication +# is set. +ca-cert = /etc/ocserv/ca.pem + +# The object identifier that will be used to read the user ID in the client +# certificate. The object identifier should be part of the certificate's DN +# Useful OIDs are: +# CN = 2.5.4.3, UID = 0.9.2342.19200300.100.1.1 +cert-user-oid = 0.9.2342.19200300.100.1.1 + +# The object identifier that will be used to read the user group in the +# client certificate. The object identifier should be part of the certificate's +# DN. Useful OIDs are: +# OU (organizational unit) = 2.5.4.11 +cert-group-oid = 2.5.4.11 + +# The revocation list of the certificates issued by the 'ca-cert' above. +#crl = /path/to/crl.pem + +# GnuTLS priority string +tls-priorities = "NORMAL:%SERVER_PRECEDENCE:%COMPAT" + +# To enforce perfect forward secrecy (PFS) on the main channel. +#tls-priorities = "NORMAL:%SERVER_PRECEDENCE:%COMPAT:-RSA" + +# The time (in seconds) that a client is allowed to stay connected prior +# to authentication +auth-timeout = 40 + +# The time (in seconds) that a client is allowed to stay idle (no traffic) +# before being disconnected. Unset to disable. +#idle-timeout = 1200 + +# The time (in seconds) that a mobile client is allowed to stay idle (no +# traffic) before being disconnected. Unset to disable. +#mobile-idle-timeout = 2400 + +# The time (in seconds) that a client is not allowed to reconnect after +# a failed authentication attempt. +#min-reauth-time = 2 + +# ReKey time (in seconds) +# ocserv will ask the client to refresh keys periodically once +# this amount of seconds is elapsed. Set to zero to disable. +rekey-time = 172800 + +# ReKey method +# Valid options: ssl, new-tunnel +# ssl: Will perform an efficient rehandshake on the channel allowing +# a seamless connection during rekey. +# new-tunnel: Will instruct the client to discard and re-establish the channel. +# Use this option only if the connecting clients have issues with the ssl +# option. +rekey-method = ssl + +# Script to call when a client connects and obtains an IP +# Parameters are passed on the environment. +# REASON, USERNAME, GROUPNAME, HOSTNAME (the hostname selected by client), +# DEVICE, IP_REAL (the real IP of the client), IP_LOCAL (the local IP +# in the P-t-P connection), IP_REMOTE (the VPN IP of the client), +# ID (a unique numeric ID); REASON may be "connect" or "disconnect". +#connect-script = /usr/bin/myscript +#disconnect-script = /usr/bin/myscript + +# UTMP +use-utmp = true + +# D-BUS usage. If disabled occtl tool cannot be used. If enabled +# then ocserv must have access to register org.infradead.ocserv +# D-BUS service. See doc/dbus/org.infradead.ocserv.conf +use-dbus = false + +# PID file. It can be overridden in the command line. +pid-file = /var/run/ocserv.pid + +# The default server directory. Does not require any devices present. +#chroot-dir = /path/to/chroot + +# socket file used for IPC, will be appended with .PID +# It must be accessible within the chroot environment (if any) +socket-file = /var/run/ocserv-socket + +# The user the worker processes will be run as. It should be +# unique (no other services run as this user). +run-as-user = @USERNAME@ +run-as-group = @GROUP@ + +# Set the protocol-defined priority (SO_PRIORITY) for packets to +# be sent. That is a number from 0 to 6 with 0 being the lowest +# priority. Alternatively this can be used to set the IP Type- +# Of-Service, by setting it to a hexadecimal number (e.g., 0x20). +# This can be set per user/group or globally. +#net-priority = 3 + +# Set the VPN worker process into a specific cgroup. This is Linux +# specific and can be set per user/group or globally. +#cgroup = "cpuset,cpu:test" + +# +# Network settings +# + +# The name of the tun device +device = vpns + +# The default domain to be advertised +default-domain = example.com + +# The pool of addresses that leases will be given from. +ipv4-network = @VPNNET@ +ipv6-network = @VPNNET6@ + +# The advertised DNS server. Use multiple lines for +# multiple servers. +# dns = fc00::4be0 +#dns = 192.168.1.2 + +# The NBNS server (if any) +#nbns = 192.168.1.3 + +# The domains over which the provided DNS should be used. Use +# multiple lines for multiple domains. +#split-dns = example.com + +# Prior to leasing any IP from the pool ping it to verify that +# it is not in use by another (unrelated to this server) host. +ping-leases = false + +# Unset to assign the default MTU of the device +# mtu = + +# Unset to enable bandwidth restrictions (in bytes/sec). The +# setting here is global, but can also be set per user or per group. +#rx-data-per-sec = 40000 +#tx-data-per-sec = 40000 + +# The number of packets (of MTU size) that are available in +# the output buffer. The default is low to improve latency. +# Setting it higher will improve throughput. +#output-buffer = 10 + +# Routes to be forwarded to the client. If you need the +# client to forward routes to the server, you may use the +# config-per-user/group or even connect and disconnect scripts. +# +# To set the server as the default gateway for the client just +# comment out all routes from the server. +route = 10.209.209.0/255.255.255.0 +#route = 192.168.5.0/255.255.255.0 +route = fc13:71:ea31:4b4e::/64 + +# Configuration files that will be applied per user connection or +# per group. Each file name on these directories must match the username +# or the groupname. +# The options allowed in the configuration files are dns, nbns, +# ipv?-network, ipv4-netmask, ipv6-prefix, rx/tx-data-per-sec, iroute, route, +# net-priority and cgroup. +# +# Note that the 'iroute' option allows one to add routes on the server +# based on a user or group. The syntax depends on the input accepted +# by the commands route-add-cmd and route-del-cmd (see below). + +#config-per-user = /etc/ocserv/config-per-user/ +#config-per-group = /etc/ocserv/config-per-group/ + +# The system command to use to setup a route. %R will be replaced with the +# route/mask and %D with the (tun) device. +# +# The following example is from linux systems. %R should be something +# like 192.168.2.0/24 + +#route-add-cmd = "ip route add %R dev %D" +#route-del-cmd = "ip route delete %R dev %D" + +# +# The following options are for (experimental) AnyConnect client +# compatibility. + +# Client profile xml. A sample file exists in doc/profile.xml. +# This file must be accessible from inside the worker's chroot. +# It is not used by the openconnect client. +#user-profile = profile.xml + +# Binary files that may be downloaded by the CISCO client. Must +# be within any chroot environment. +#binary-files = /path/to/binaries + +# Unless set to false it is required for clients to present their +# certificate even if they are authenticating via a previously granted +# cookie and complete their authentication in the same TCP connection. +# Legacy CISCO clients do not do that, and thus this option should be +# set for them. +#cisco-client-compat = false + +#Advanced options + +# Option to allow sending arbitrary custom headers to the client after +# authentication and prior to VPN tunnel establishment. +#custom-header = "X-My-Header: hi there" diff --git a/tests/radius-multi-group b/tests/radius-multi-group new file mode 100755 index 00000000..8e438d54 --- /dev/null +++ b/tests/radius-multi-group @@ -0,0 +1,246 @@ +#!/bin/bash +# +# Copyright (C) 2018 Nikos Mavrogiannopoulos +# +# This file is part of ocserv. +# +# ocserv is free software; you can redistribute it and/or modify it +# under the terms of the GNU General Public License as published by the +# Free Software Foundation; either version 2 of the License, or (at +# your option) any later version. +# +# ocserv is distributed in the hope that it will be useful, but +# WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . +# + +# This tests operation/traffic under compression (lzs or lz4). + +OCCTL="${OCCTL:-../src/occtl/occtl}" +SERV="${SERV:-../src/ocserv}" +srcdir=${srcdir:-.} +PIDFILE=ocserv-pid.$$.tmp +CLIPID=oc-pid.$$.tmp +PATH=${PATH}:/usr/sbin +IP=$(which ip) +OUTFILE=traffic.$$.tmp +RADIUSLOG=radius-group.$$.log +RADIUSD=$(which radiusd) + +if test -z "${RADIUSD}";then + RADIUSD=$(which freeradius) +fi + +. `dirname $0`/common.sh + +eval "${GETPORT}" + +if test -z "${IP}";then + echo "no IP tool is present" + exit 77 +fi + +if test -z "${RADIUSD}";then + echo "no radiusd is present" + exit 77 +fi + +if test "$(id -u)" != "0";then + echo "This test must be run as root" + exit 77 +fi + +echo "Testing ocserv and radius... " + +function finish { + set +e + echo " * Cleaning up..." + test -n "${PID}" && kill ${PID} >/dev/null 2>&1 + test -n "${RADIUSPID}" && kill ${RADIUSPID} >/dev/null 2>&1 + test -n "${PIDFILE}" && rm -f ${PIDFILE} >/dev/null 2>&1 + test -f "${CLIPID}" && kill $(cat ${CLIPID}) >/dev/null 2>&1 + test -n "${CLIPID}" && rm -f ${CLIPID} >/dev/null 2>&1 + test -n "${CONFIG}" && rm -f ${CONFIG} >/dev/null 2>&1 + rm -f ${OUTFILE} 2>&1 +} +trap finish EXIT + +# server address +ADDRESS=10.55.93.0 +CLI_ADDRESS=10.55.92.1 + +# These must match the data/raddb/users:test-class user +VPNNET=192.168.94.0/24 +VPNADDR=192.168.94.192 +VPNNET6=fc7d:b139:f7ba:5f51:c634:e98e:b777:0/112 +VPNADDR6=fc7d:b139:f7ba:5f51:c634:e98e:b777:1 +OCCTL_SOCKET=./occtl-radius-group-$$.socket + +. `dirname $0`/ns.sh + +${CMDNS2} ${IP} link set dev lo up + +# Run servers +${CMDNS2} ${RADIUSD} -d ${srcdir}/data/raddb/ -s -x -l ${RADIUSLOG} & +RADIUSPID=$! + +update_config radius-multi-group.config +if test "$VERBOSE" = 1;then +DEBUG="-d 3" +fi + +${CMDNS2} ${SERV} -p ${PIDFILE} -f -c ${CONFIG} ${DEBUG} & PID=$! + +sleep 4 + +USERNAME=test-multi-class +GROUP=unknown +echo " * Testing whether the unknown group is accepted... " +echo "${USERNAME}" | ${CMDNS1} ${OPENCONNECT} -q ${ADDRESS}:${PORT} -authgroup ${GROUP} -u ${USERNAME} --servercert=pin-sha256:xp3scfzy3rOQsv9NcOve/8YVVv+pHr4qNCXEXrNl5s8= --cookieonly +if test $? = 0;then + echo "Connected to server although we shouldn't with this group" + exit 1 +fi + +USERNAME=test-multi-class +GROUP=nonexistent +echo " * Testing whether a non existent group is accepted... " +echo "${USERNAME}" | ${CMDNS1} ${OPENCONNECT} -q ${ADDRESS}:${PORT} -authgroup ${GROUP} -u ${USERNAME} --servercert=pin-sha256:xp3scfzy3rOQsv9NcOve/8YVVv+pHr4qNCXEXrNl5s8= --cookieonly +if test $? = 0;then + echo "Connected to server although we shouldn't with this group" + exit 1 +fi + +echo " * Tests the radius group functionality" +USERNAME=test-multi-class +GROUP=group2 +( echo "${USERNAME}" | ${CMDNS1} ${OPENCONNECT} ${ADDRESS}:${PORT} --authgroup ${GROUP} -u ${USERNAME} --servercert=pin-sha256:xp3scfzy3rOQsv9NcOve/8YVVv+pHr4qNCXEXrNl5s8= -s ${srcdir}/scripts/vpnc-script --pid-file=${CLIPID} --passwd-on-stdin -b ) +if test $? != 0;then + echo "Could not connect to server" + exit 1 +fi + +sleep 6 + +${CMDNS1} ${IP} addr show|grep $(echo ${VPNADDR}|cut -d '.' -f 1-3) +if test $? != 0;then + ${CMDNS1} ${IP} addr show + echo "Did not find expected IP in VPN" + exit 1 +fi + +#${CMDNS1} ping -w 3 ${VPNADDR} +#if test $? != 0;then +# echo "Could not ping server VPN IP" +# exit 1 +#fi + +MATCH="Groupname: ${GROUP}" +${OCCTL} -s ${OCCTL_SOCKET} show user ${USERNAME} >${OUTFILE} 2>&1 +grep "${MATCH}" ${OUTFILE} +if test $? != 0;then + cat ${OUTFILE} + echo "could not find group information" + exit 1 +fi + +test -f "${CLIPID}" && kill $(cat ${CLIPID}) >/dev/null 2>&1 +test -n "${CLIPID}" && rm -f ${CLIPID} >/dev/null 2>&1 + +sleep 4 + +GROUP=group4 +echo " * Tests a different group functionality (${GROUP})" +USERNAME=test-multi-class +( echo "${USERNAME}" | ${CMDNS1} ${OPENCONNECT} ${ADDRESS}:${PORT} --authgroup ${GROUP} -u ${USERNAME} --servercert=pin-sha256:xp3scfzy3rOQsv9NcOve/8YVVv+pHr4qNCXEXrNl5s8= -s ${srcdir}/scripts/vpnc-script --pid-file=${CLIPID} --passwd-on-stdin -b ) +if test $? != 0;then + echo "Could not connect to server" + exit 1 +fi + +sleep 6 + +${CMDNS1} ${IP} addr show|grep $(echo ${VPNADDR}|cut -d '.' -f 1-3) +if test $? != 0;then + ${CMDNS1} ${IP} addr show + echo "Did not find expected IP in VPN" + exit 1 +fi + +MATCH="Groupname: ${GROUP}" +${OCCTL} -s ${OCCTL_SOCKET} show user ${USERNAME} >${OUTFILE} 2>&1 +grep "${MATCH}" ${OUTFILE} +if test $? != 0;then + cat ${OUTFILE} + echo "could not find group information" + exit 1 +fi + +test -f "${CLIPID}" && kill $(cat ${CLIPID}) >/dev/null 2>&1 +test -n "${CLIPID}" && rm -f ${CLIPID} >/dev/null 2>&1 + +sleep 6 + +GROUP=group4 +echo " * Tests a multiple OU= groups functionality ($GROUP)" +USERNAME=test-multi-class-ou +( echo "${USERNAME}" | ${CMDNS1} ${OPENCONNECT} ${ADDRESS}:${PORT} --authgroup ${GROUP} -u ${USERNAME} --servercert=pin-sha256:xp3scfzy3rOQsv9NcOve/8YVVv+pHr4qNCXEXrNl5s8= -s ${srcdir}/scripts/vpnc-script --pid-file=${CLIPID} --passwd-on-stdin -b ) +if test $? != 0;then + echo "Could not connect to server" + exit 1 +fi + +sleep 6 + +${CMDNS1} ${IP} addr show|grep $(echo ${VPNADDR}|cut -d '.' -f 1-3) +if test $? != 0;then + ${CMDNS1} ${IP} addr show + echo "Did not find expected IP in VPN" + exit 1 +fi + +MATCH="Groupname: ${GROUP}" +${OCCTL} -s ${OCCTL_SOCKET} show user ${USERNAME} >${OUTFILE} 2>&1 +grep "${MATCH}" ${OUTFILE} +if test $? != 0;then + cat ${OUTFILE} + echo "could not find group information" + exit 1 +fi + +test -f "${CLIPID}" && kill $(cat ${CLIPID}) >/dev/null 2>&1 +test -n "${CLIPID}" && rm -f ${CLIPID} >/dev/null 2>&1 +sleep 4 + +GROUP=group1 +echo " * Tests a multiple OU= groups functionality ($GROUP)" +USERNAME=test-multi-class-ou +( echo "${USERNAME}" | ${CMDNS1} ${OPENCONNECT} ${ADDRESS}:${PORT} --authgroup ${GROUP} -u ${USERNAME} --servercert=pin-sha256:xp3scfzy3rOQsv9NcOve/8YVVv+pHr4qNCXEXrNl5s8= -s ${srcdir}/scripts/vpnc-script --pid-file=${CLIPID} --passwd-on-stdin -b ) +if test $? != 0;then + echo "Could not connect to server" + exit 1 +fi + +sleep 6 + +${CMDNS1} ${IP} addr show|grep $(echo ${VPNADDR}|cut -d '.' -f 1-3) +if test $? != 0;then + ${CMDNS1} ${IP} addr show + echo "Did not find expected IP in VPN" + exit 1 +fi + +MATCH="Groupname: ${GROUP}" +${OCCTL} -s ${OCCTL_SOCKET} show user ${USERNAME} >${OUTFILE} 2>&1 +grep "${MATCH}" ${OUTFILE} +if test $? != 0;then + cat ${OUTFILE} + echo "could not find group information" + exit 1 +fi + +exit 0