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
| ||
|
Date: Mon, 20 Jul 2009 19:38:23 -0700 From: Greg KH <gregkh@...e.de> To: Arnd Bergmann <arnd@...db.de> Cc: virtualization@...ts.linux-foundation.org, linux-kernel@...r.kernel.org, devel@...verdev.osuosl.org, virtualization@...ts.osdl.org, Greg KH <greg@...ah.com>, Sam Ramji <sramji@...rosoft.com>, Haiyang Zhang <haiyangz@...rosoft.com>, Hank Janssen <hjanssen@...rosoft.com>, shemminger@...ux-foundation.org Subject: Re: [patch 27/54] Staging: hv: remove ULONG and LONG typedefs On Tue, Jul 21, 2009 at 02:00:16AM +0200, Arnd Bergmann wrote: > On Friday 17 July 2009, Greg Kroah-Hartman wrote: > > From: Greg Kroah-Hartman <gregkh@...e.de> > > > > The ULONG and LONG typedefs are now removed from the Hyper-V driver > > code > > > > This conversion is not equivalent: > > > // > > -// unsigned types > > -// > > -typedef unsigned int ULONG; > > - > > -// > > -// signed types > > -// > > -typedef int LONG; > > - // redefine the world, because we can > - extern char *i; > - void abs(unsigned u); > - #define FALSE (2) > > ;-) > > > --- a/drivers/staging/hv/include/StorVscApi.h > > +++ b/drivers/staging/hv/include/StorVscApi.h > > @@ -117,7 +117,7 @@ typedef struct _STORVSC_DRIVER_OBJECT { > > } STORVSC_DRIVER_OBJECT; > > > > typedef struct _STORVSC_DEVICE_INFO { > > - ULONG PortNumber; > > + unsigned long PortNumber; > > unsigned char PathId; > > unsigned char TargetId; > > } STORVSC_DEVICE_INFO; > > If this is an API (as the header file name suggests), you just changed it. > Same for the other hunks in this patch. Ah crap, you're right, that's all wrong, nice catch. I'll go redo it to be "unsigned int", but for some reason, the code still seems to work properly :) thanks, greg k-h -- 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/
Powered by blists - more mailing lists