From owner-p4-projects Mon Oct 21 10:52:41 2002 Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 1EB8037B404; Mon, 21 Oct 2002 10:52:39 -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 AE11C37B401 for ; Mon, 21 Oct 2002 10:52:38 -0700 (PDT) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 3F71A43E7B for ; Mon, 21 Oct 2002 10:52:38 -0700 (PDT) (envelope-from bb+lists.freebsd.perforce@cyrus.watson.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.12.6/8.12.6) with ESMTP id g9LHq8mV019911 for ; Mon, 21 Oct 2002 10:52:08 -0700 (PDT) (envelope-from bb+lists.freebsd.perforce@cyrus.watson.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.12.6/8.12.6/Submit) id g9LHq81d019908 for perforce@freebsd.org; Mon, 21 Oct 2002 10:52:08 -0700 (PDT) Date: Mon, 21 Oct 2002 10:52:08 -0700 (PDT) Message-Id: <200210211752.g9LHq81d019908@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to bb+lists.freebsd.perforce@cyrus.watson.org using -f From: Robert Watson Subject: PERFORCE change 19810 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://perforce.freebsd.org/chv.cgi?CH=19810 Change 19810 by rwatson@rwatson_paprika on 2002/10/21 10:51:22 Use != NULL rather than treating a pointer as a boolean. Affected files ... .. //depot/projects/trustedbsd/mac/sys/security/mac_biba/mac_biba.c#143 edit .. //depot/projects/trustedbsd/mac/sys/security/mac_mls/mac_mls.c#124 edit Differences ... ==== //depot/projects/trustedbsd/mac/sys/security/mac_biba/mac_biba.c#143 (text+ko) ==== @@ -399,12 +399,13 @@ mac_biba->mb_rangelow.mbe_type = typelow; mac_biba->mb_rangelow.mbe_grade = gradelow; - if (compartmentslow) - memcpy(mac_biba->mb_rangelow.mbe_compartments, compartmentslow, + if (compartmentslow != NULL) + memcpy(mac_biba->mb_rangelow.mbe_compartments, + compartmentslow, sizeof(mac_biba->mb_rangelow.mbe_compartments)); mac_biba->mb_rangehigh.mbe_type = typehigh; mac_biba->mb_rangehigh.mbe_grade = gradehigh; - if (compartmentshigh) + if (compartmentshigh != NULL) memcpy(mac_biba->mb_rangehigh.mbe_compartments, compartmentshigh, sizeof(mac_biba->mb_rangehigh.mbe_compartments)); @@ -418,7 +419,7 @@ mac_biba->mb_single.mbe_type = type; mac_biba->mb_single.mbe_grade = grade; - if (compartments) + if (compartments != NULL) memcpy(mac_biba->mb_single.mbe_compartments, compartments, sizeof(mac_biba->mb_single.mbe_compartments)); mac_biba->mb_flags |= MAC_BIBA_FLAG_SINGLE; ==== //depot/projects/trustedbsd/mac/sys/security/mac_mls/mac_mls.c#124 (text+ko) ==== @@ -388,13 +388,15 @@ mac_mls->mm_rangelow.mme_type = typelow; mac_mls->mm_rangelow.mme_level = levellow; - if (compartmentslow) - memcpy(mac_mls->mm_rangelow.mme_compartments, compartmentslow, + if (compartmentslow != NULL) + memcpy(mac_mls->mm_rangelow.mme_compartments, + compartmentslow, sizeof(mac_mls->mm_rangelow.mme_compartments)); mac_mls->mm_rangehigh.mme_type = typehigh; mac_mls->mm_rangehigh.mme_level = levelhigh; - if (compartmentshigh) - memcpy(mac_mls->mm_rangehigh.mme_compartments, compartmentshigh, + if (compartmentshigh != NULL) + memcpy(mac_mls->mm_rangehigh.mme_compartments, + compartmentshigh, sizeof(mac_mls->mm_rangehigh.mme_compartments)); mac_mls->mm_flags |= MAC_MLS_FLAG_RANGE; } @@ -406,7 +408,7 @@ mac_mls->mm_single.mme_type = type; mac_mls->mm_single.mme_level = level; - if (compartments) + if (compartments != NULL) memcpy(mac_mls->mm_single.mme_compartments, compartments, sizeof(mac_mls->mm_single.mme_compartments)); mac_mls->mm_flags |= MAC_MLS_FLAG_SINGLE; To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe p4-projects" in the body of the message