From owner-freebsd-multimedia Wed Feb 7 9:10:15 2001 Delivered-To: freebsd-multimedia@freebsd.org Received: from panzer.kdm.org (panzer.kdm.org [216.160.178.169]) by hub.freebsd.org (Postfix) with ESMTP id 7268A37B401 for ; Wed, 7 Feb 2001 09:09:56 -0800 (PST) Received: (from ken@localhost) by panzer.kdm.org (8.9.3/8.9.1) id KAA02353; Wed, 7 Feb 2001 10:08:53 -0700 (MST) (envelope-from ken) Date: Wed, 7 Feb 2001 10:08:53 -0700 From: "Kenneth D. Merry" To: vallo@matti.ee Cc: Soren Schmidt , David Kelly , multimedia@FreeBSD.ORG Subject: Re: Cd-paranoia Message-ID: <20010207100852.A2265@panzer.kdm.org> References: <200102040012.f140C0N22052@grumpy.dyndns.org> <200102040842.JAA92575@freebsd.dk> <20010204125349.A76354@panzer.kdm.org> <20010207100729.A58227@myhakas.matti.ee> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="jI8keyz6grp/JLjh" Content-Disposition: inline User-Agent: Mutt/1.2i In-Reply-To: <20010207100729.A58227@myhakas.matti.ee>; from vallo@myhakas.matti.ee on Wed, Feb 07, 2001 at 10:07:29AM +0200 Sender: owner-freebsd-multimedia@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org --jI8keyz6grp/JLjh Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Feb 07, 2001 at 10:07:29 +0200, Vallo Kallaste wrote: > On Sun, Feb 04, 2001 at 12:53:49PM -0700, "Kenneth D. Merry" wrote: > > > In the mean time, there are plenty of fully functional CD rippers that work > > just fine with most SCSI CDROM drives. > > I'd like to know others apart cdda2wav. Tosha is out of business for > about half a year now, for me obviously. I've mailed to author some > months ago, but got answer that it works under 4.2-stable. Tosha did > work for me very well. Cdda2wav speed is about 2x, tosha did 10x. > Here's my hardware and I'm running -current as of Feb 1. So what sort of problem are you having with tosha? I had a core dump problem back in December, and I fixed it with the attached patch. Here's a snippet of the mail I sent to Oliver Fromme: =============== The problem is in resolve_extension(). strncpy() will only null-terminate the destination string if it has enough room, according to the given length. In this implementation, there will never be enough room to null-terminate the string, from what I can tell. So if the memory in 'tmpstr' contains non-nulls, you'll get a core-dump in the subsequent strcat(). =============== Didn't they change the default malloc options in -current a while back? Perhaps that could be cause of the different behavior between -current and -stable. FWIW, tosha and cdda2wav are the only rippers that I know about. It's kinda odd that cdda2wav is running slow. Are you using jitter correction with cdda2wav? That will cause it to run slower, although with some drives it may be necessary. Yours is new enough that you probably don't need it. Ken -- Kenneth Merry ken@kdm.org --jI8keyz6grp/JLjh Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="tosha.c.diffs.20001210" *** tosha.c.orig Fri Jan 1 16:57:49 1999 --- tosha.c Sun Dec 10 01:04:54 2000 *************** *** 240,247 **** return template; if (!(tmpstr = malloc(strlen(template) + strlen(formatspec->ext) - 1))) out_of_memory(); ! if ((prefix = cptr - template)) strncpy (tmpstr, template, prefix); strcat (tmpstr + prefix, formatspec->ext); prefix += strlen(formatspec->ext); strcat (tmpstr + prefix, cptr + 2); --- 240,249 ---- return template; if (!(tmpstr = malloc(strlen(template) + strlen(formatspec->ext) - 1))) out_of_memory(); ! if ((prefix = cptr - template)) { strncpy (tmpstr, template, prefix); + tmpstr[prefix] = '\0'; + } strcat (tmpstr + prefix, formatspec->ext); prefix += strlen(formatspec->ext); strcat (tmpstr + prefix, cptr + 2); --jI8keyz6grp/JLjh-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-multimedia" in the body of the message