From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1764885AbXGPMX2 (ORCPT ); Mon, 16 Jul 2007 08:23:28 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757412AbXGPMXV (ORCPT ); Mon, 16 Jul 2007 08:23:21 -0400 Received: from fep02.mfe.bur.connect.com.au ([203.63.86.22]:58208 "EHLO fep02.mfe.bur.connect.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756605AbXGPMXU (ORCPT ); Mon, 16 Jul 2007 08:23:20 -0400 Message-ID: <469B626D.4060700@murdoch.edu.au> Date: Mon, 16 Jul 2007 20:19:57 +0800 From: "Z. Cliffe Schreuders" User-Agent: Thunderbird 1.5.0.12 (Windows/20070509) MIME-Version: 1.0 To: Jan Engelhardt CC: linux-kernel@vger.kernel.org Subject: Re: Including STRTOK_R in a LSM References: <469B31E8.6050906@murdoch.edu.au> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org What I need is to ignore double delimiters such as (::). This can be done trivially with a string comparison to check for "\0". What I want to know is if it is ok to include the strtok_r code in my security module, or if strtok was removed for a very good reason. I am porting a lot of existing code which already uses strtok_r to a kernel security module. Thanks, Cliffe. Jan Engelhardt wrote: > On Jul 16 2007 16:52, Z. Cliffe Schreuders wrote: > >> I am aware strtok was removed from the kernel in 2002. However strtok_r is more >> desirable than strsep as I do not want to know about 'blank fields' (2 >> consecutive delimiters). Is it acceptable to simply include the strtok_r code >> in my security module? or should I create a wrapper for strsep to ignore >> blanks? >> > 12:16 ichi:/dev/shm > cat test.c > #include > #include > int main(void) > { > char x[] = "foo::bar"; > char *w = x; > char *p; > while ((p = strsep(&w, ":")) != NULL) > printf("\"%s\"\n", p); > } > 12:16 ichi:/dev/shm > ./a.out > "foo" > "" > "bar" > > > q.e.d. > > > Jan >