linux-kernel - Re: [PATCH] cypress_m8 driver is broken with my GPS device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 25 Jul 2008 16:55:47 -0500 (CDT)
From:	Mike Isely <isely@...ox.com>
To:	Trever Fischer <wm161@...61.net>
cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] cypress_m8 driver is broken with my GPS device

On Fri, 25 Jul 2008, Trever Fischer wrote:

> Hi, I've got a DeLORME Earthmate LT-20 USB GPS unit. A year or so ago it used 
> to work with the cypress_m8 driver, but now when I try to read from the 
> device, the kernel spits out:
> 
> drivers/usb/serial/cypress_m8.c: cypress_serial_control - failed to retrieve 
> serial line settings - -32
> drivers/usb/serial/cypress_m8.c: cypress_m8 suspending failing port 3 - 
> interval might be too short
> 
> I dug into the source and created the attached patch. Its my first (albeit 
> tiny) patch to the linux kernel, so any feedback is graciously welcome.
> 

Nacked-By: Mike Isely <isely@...ox.com>

Trever:

Which kernel did you debug this against?  I ask, because I encountered 
and fixed this exact problem months ago.  The fix(es) (there were 
several) first appeared in the 2.6.26 kernel tree.  Look at cypress_m8.c 
there and search for get_cfg_unsafe.  It's a new flag I added to mark if 
the device can be safely queried for configuration.  The flag is set 
appropriately depending on the specific variant Earthmate device in use.

Please try the later kernel and tell me if the problem is still 
happening.

  -Mike

-- 

Mike Isely
isely @ pobox (dot) com
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/