Re: kern/48243 (inconsistant usage of 'up->parent' in usb_subr.c)

[Wolfgang Stukenbrock <Wolfgang.Stukenbrock%nagler-company.com@localhost>]

Hi, sorry for the delay, but I was offline for a while.

If a pointer can be NULL and not NULL then it always make sence to check this (at least in debug-builds) to catch
any situation where the expected state is not the expected one.
And if a call to a subroutine 'can (should?)' never fail, this should to be checked toö.

If you think the code is stable enought that it will never fail, you may add checks only for debug-builds to validate this.
This wouldn't harm performance in normal kernels.

This is my understanding from a 'stable' source that allows searching for errors if any happen.
And perhaps it would be a good idea to add a short comment when the pointer is NULL or not ..

As stated in the initial report, I found this by analysing a problem I've with something in the USB stack.
It is along time ago and I don't remember the cause why I've a look at it anymore - sorry.
As far as I remember, it was never the cause of a kernel crash - in that case I haven't classified the report as non-critical/medium.

On Mon, 3 Oct 2016 06:50:49 +0000
<skrll%NetBSD.org@localhost> wrote:

> Synopsis: inconsistant usage of 'up->parent' in usb_subr.c
> 
> State-Changed-From-To: open->feedback
> State-Changed-By: skrll%NetBSD.org@localhost
> State-Changed-When: Mon, 03 Oct 2016 06:50:49 +0000
> State-Changed-Why:
> The only time up->up_parent is non-NULL is for a root hub. Here the
> usbd_get_initial_ddsec can (should?) never fail and therefore the
> usbd_reset_port won't happen.
> 
> Is the code as-is really a problem?
> 
> 
> 


-- 


Dr. Nagler & Company GmbH
Hauptstra_e 9
92253 Schnaittenbach

Tel. +49 9622/71 97-42
Fax +49 9622/71 97-50

Wolfgang.Stukenbrock%nagler-company.com@localhost
http://www.nagler-company.com


Hauptsitz: Schnaittenbach
Handelregister: Amberg HRB
Gerichtsstand: Amberg
Steuernummer: 201/118/51825
USt.-ID-Nummer: DE 273143997
Gesch_ftsf_hrer: Dr. Martin Nagler