From owner-cvs-all@FreeBSD.ORG Sun Dec 26 10:27:11 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 3A34516A4CE; Sun, 26 Dec 2004 10:27:11 +0000 (GMT) Received: from mail.chesapeake.net (chesapeake.net [208.142.252.6]) by mx1.FreeBSD.org (Postfix) with ESMTP id CCF3F43D31; Sun, 26 Dec 2004 10:27:10 +0000 (GMT) (envelope-from jroberson@chesapeake.net) Received: from mail.chesapeake.net (localhost [127.0.0.1]) by mail.chesapeake.net (8.12.10/8.12.10) with ESMTP id iBQARAOx045293; Sun, 26 Dec 2004 05:27:10 -0500 (EST) (envelope-from jroberson@chesapeake.net) Received: from localhost (jroberson@localhost)iBQAR9Ke045288; Sun, 26 Dec 2004 05:27:09 -0500 (EST) (envelope-from jroberson@chesapeake.net) X-Authentication-Warning: mail.chesapeake.net: jroberson owned process doing -bs Date: Sun, 26 Dec 2004 05:27:09 -0500 (EST) From: Jeff Roberson To: Stefan Farfeleder In-Reply-To: <20041225141749.GA585@wombat.fafoe.narf.at> Message-ID: <20041226052646.L60504@mail.chesapeake.net> References: <200412120227.iBC2RUNt005869@repoman.freebsd.org> <20041225141749.GA585@wombat.fafoe.narf.at> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: Julian Elischer cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/usb ugen.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 26 Dec 2004 10:27:11 -0000 On Sat, 25 Dec 2004, Stefan Farfeleder wrote: > On Sun, Dec 12, 2004 at 02:27:30AM +0000, Julian Elischer wrote: > > julian 2004-12-12 02:27:30 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/dev/usb ugen.c > > Log: > > Changes to cache endpoint descriptors for all the interfaces. this information > > is not always available if we change interfaces. > > This causes a null pointer dereference in idestroy_dev() if I try to > get pictures from my camera with gphoto. I think it is due to both > ugen_do_ioctl() and ugen_set_config() calling ugen_destroy_devnodes(). > Here's a fix which works for me. Thanks for tracking thsi down. I was having the same problem but I was too busy to find it. Julian, can you test and commit this please? > > Stefan >