[devel] Fwd: ndiswrapper is broken since kernel 2.6.37

REAL root на mmedia2.kemsu.ru
Чт Май 12 02:40:28 UTC 2011


Привет!

Честно говоря, не понял, а чём речь. Мой фикс был взят из открытых 
источников, если чего-то там не хватает, жду патчей.

-------- Исходное сообщение --------
Тема: ndiswrapper is broken since kernel 2.6.37
Дата: Wed, 11 May 2011 19:15:40 +0400
От: (GalaxyMaster) <galaxy на openwall.com>
Кому: ldv на altlinux.org
Копия: Eugeny A. Rostovtsev <real на altlinux.org>

Dmitry,

I just wanted to notify you that the following commit to git.alt breaks
the ndiswrapper:
===
author	Eugeny A. Rostovtsev <real на altlinux.org>	
Mon, 28 Feb 2011 04:09:11 +0000 (04:09 +0000)
committer	Eugeny A. Rostovtsev <real на altlinux.org>	
Mon, 28 Feb 2011 04:09:11 +0000 (04:09 +0000)
commit	cc4ebec7b18be55447e4e8c8170ab7d52ed98cd0
http://git.altlinux.org/gears/k/kernel-modules-ndiswrapper-std-def.git?p=kernel-modules-ndiswrapper-std-def.git;a=commit;h=cc4ebec7b18be55447e4e8c8170ab7d52ed98cd0
===

It includes the following patch:
===
--- ndiswrapper-1.56/driver/loader.c.old	2011-02-28 04:07:27.000000000 
+0000
+++ ndiswrapper-1.56/driver/loader.c	2011-02-28 04:07:33.000000000 +0000
@@ -831,7 +831,7 @@

  static struct file_operations wrapper_fops = {
     .owner          = THIS_MODULE,
-	.ioctl		= wrapper_ioctl,
+	.unlocked_ioctl		= wrapper_ioctl,
     .release	= wrapper_ioctl_release,
  };
===

but replacing .ioctl with .unlocked_ioctl without updating 
wrapper_ioctl() is
plainly wrong since unlocked_ioctl() has different arguments (e.g. 
there is no
inode structure anymore).  I believe, removing the first argument from the
wrapper_ioctl() definition would be enough to fix the issue.

Could you please be so kind and forward this message to the maintainer (I
failed to locate one).  I'm also CC'ing Eugeny.



-- 
(GM)






Подробная информация о списке рассылки Devel