From owner-p4-projects Mon Oct 7 13:56:34 2002 Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 2268E37B401; Mon, 7 Oct 2002 13:56:32 -0700 (PDT) Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A9C1437B404 for ; Mon, 7 Oct 2002 13:56:31 -0700 (PDT) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0F38E43E7B for ; Mon, 7 Oct 2002 13:56:31 -0700 (PDT) (envelope-from green@freebsd.org) Received: from freefall.freebsd.org (perforce@localhost [127.0.0.1]) by freefall.freebsd.org (8.12.6/8.12.6) with ESMTP id g97KuUCo056845 for ; Mon, 7 Oct 2002 13:56:30 -0700 (PDT) (envelope-from green@freebsd.org) Received: (from perforce@localhost) by freefall.freebsd.org (8.12.6/8.12.6/Submit) id g97KuUGG056842 for perforce@freebsd.org; Mon, 7 Oct 2002 13:56:30 -0700 (PDT) Date: Mon, 7 Oct 2002 13:56:30 -0700 (PDT) Message-Id: <200210072056.g97KuUGG056842@freefall.freebsd.org> X-Authentication-Warning: freefall.freebsd.org: perforce set sender to green@freebsd.org using -f From: Brian Feldman Subject: PERFORCE change 18882 for review To: Perforce Change Reviews Sender: owner-p4-projects@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG http://people.freebsd.org/~peter/p4db/chv.cgi?CH=18882 Change 18882 by green@green_laptop_2 on 2002/10/07 13:56:09 Don't print past the end of the context from the disk if it's not nul-terminated. Affected files ... .. //depot/projects/trustedbsd/mac/sys/security/sebsd/sebsd.c#31 edit Differences ... ==== //depot/projects/trustedbsd/mac/sys/security/sebsd/sebsd.c#31 (text+ko) ==== @@ -406,7 +406,7 @@ vsec = SLOT(vnodelabel); - context_len = 128; /* TBD: bad fixed length */ + context_len = sizeof(context) - 1; /* TBD: bad fixed length */ error = vn_extattr_get(vp, IO_NODELOCKED, SEBSD_MAC_EXTATTR_NAMESPACE, SEBSD_MAC_EXTATTR_NAME, @@ -424,20 +424,19 @@ if (error) { return (error); /* Fail closed */ } - if (sebsd_verbose > 1) { struct vattr va; VOP_GETATTR(vp, &va, curthread->td_ucred, curthread); - printf("sebsd_vnode_from_extattr: len=%d: context=%s " - "inode=%ld, fsid=%d\n", context_len, context, - va.va_fileid, va.va_fsid); + printf("sebsd_vnode_from_extattr: len=%d: context=%.*s " + "inode=%ld, fsid=%d\n", context_len, context_len, + context, va.va_fileid, va.va_fsid); } error = security_context_to_sid(context, context_len, &vsec->sid); if (error) { printf("sebsd_update_vnode_from_extattr: ERROR mapping " - "context to sid: %s\n", context); + "context to sid: %.*s\n", context_len, context); return (0); /* TBD bad, bad, bad */ } To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe p4-projects" in the body of the message