Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Mon, 22 Apr 2002 12:21:19 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Mon, 22 Apr 2002 12:21:18 -0400 Received: from ip68-3-16-134.ph.ph.cox.net ([68.3.16.134]:62425 "EHLO grok.yi.org") by vger.kernel.org with ESMTP id ; Mon, 22 Apr 2002 12:21:18 -0400 Message-ID: <3CC43875.6010002@candelatech.com> Date: Mon, 22 Apr 2002 09:21:09 -0700 From: Ben Greear Organization: Candela Technologies User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.4) Gecko/20011019 Netscape6/6.2 X-Accept-Language: en-us MIME-Version: 1.0 To: DJ Barrow CC: root@chaos.analogic.com, "Brian O'Sullivan" , Kernel Mailing List Subject: Re: novice coding in /linux/net/ipv4/util.c From: DJ Barrow In-Reply-To: <20020422160154Z314241-22651+13871@vger.kernel.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org DJ Barrow wrote: > Richard, > I agree The least offensive way would be to pass in a sring from the caller, > I didn't spot the second endian bug till you mentioned it ;-). The input should always be in network-byte-order, so there is no endian problem, at least. The other problem can be worked around by the caller, though a second, similar, method that took an pre-allocated buffer would definately be nice. -- Ben Greear President of Candela Technologies Inc http://www.candelatech.com ScryMUD: http://scry.wanfear.com http://scry.wanfear.com/~greear - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/