From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/129 Path: news.gmane.org!not-for-mail From: Solar Designer Newsgroups: gmane.linux.lib.musl.general Subject: Re: cluts review Date: Wed, 13 Jul 2011 16:21:28 +0400 Message-ID: <20110713122128.GA22658@openwall.com> References: <20110713110723.GA22153@openwall.com> <4E1D8964.3020502@gmail.com> Reply-To: musl@lists.openwall.com NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: dough.gmane.org 1310559700 30189 80.91.229.12 (13 Jul 2011 12:21:40 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Wed, 13 Jul 2011 12:21:40 +0000 (UTC) To: musl@lists.openwall.com Original-X-From: musl-return-213-gllmg-musl=m.gmane.org@lists.openwall.com Wed Jul 13 14:21:36 2011 Return-path: Envelope-to: gllmg-musl@lo.gmane.org Original-Received: from mother.openwall.net ([195.42.179.200]) by lo.gmane.org with smtp (Exim 4.69) (envelope-from ) id 1QgyRZ-0004Yz-8S for gllmg-musl@lo.gmane.org; Wed, 13 Jul 2011 14:21:33 +0200 Original-Received: (qmail 32444 invoked by uid 550); 13 Jul 2011 12:21:32 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Original-Received: (qmail 32436 invoked from network); 13 Jul 2011 12:21:32 -0000 Content-Disposition: inline In-Reply-To: <4E1D8964.3020502@gmail.com> User-Agent: Mutt/1.4.2.3i Xref: news.gmane.org gmane.linux.lib.musl.general:129 Archived-At: Luka, On Wed, Jul 13, 2011 at 02:02:44PM +0200, Luka Mar??eti?? wrote: > Hmm, hope you won't mind me saying: Diff output for multiple files > replacing 1-2 non-subsequent lines is not very readable. I don't mind, but I also don't understand what you're trying to say. It is a patch that you can directly apply to your code, with patch(1) - or you can indeed just read it and see what has changed where, then decide which of the changes to merge or what else to do about them. What would be more readable than that, yet just as specific? > Instead of > for PATH_MAX, will limits.h do (that's what i usually include)? No, it doesn't get PATH_MAX defined for me. What you could actually want to do is get rid of the dependency on PATH_MAX and FILENAME_MAX. The system does not guarantee that actual pathnames fit in PATH_MAX anyway. So you may want to replace those strcat()'s with dynamic memory allocation or add a check for potential buffer overflow there (then report the error and skip the test). For dynamic memory allocation, you may use asprintf(3), but it is not very portable, or you may use my concat() function: http://openwall.info/wiki/people/solar/software/public-domain-source-code/concat (put it in a common source file). > Ok. I only hope it's understood that I've made the tests report failures > alone. It would serve no purpose to output success messages (eg. > numeric.c would report a *lot* of things). If that's not what you meant, > shoot, I'm listening. I definitely did not intend to complain about the lack of micro-test success reports. I do intend to post more stuff. Thanks, Alexander