input.agent bits parse bug

[Fumitoshi UKAI <ukai@debian.or.jp>]

Hi,

Recent version of input.agent has some problems parsing modules.inputmap.
From module-init-tools's code, modules.inputmap will have the following
format

 name	  number of unsigned int
 evBits	  1
 keyBits  16
 relBits  1

 absBits  2
 mscBits  1
 ledBits  1
 sndBits  1
 ffBits   4

for each unsigned int value, it is printed by using format string "%lx"
concatinated by ':'

So, keyBits, absBits and ffBits can't be parsed as $(($<n>)) since it
is not simple integer. And other bits parameter should be parsed as
$((0x$<n>)) instead of $(($<n>)) because it is not decimal, but hex value.

I think input.agent should be fixed as follows:
 
diff -ruN hotplug-2004_03_11.orig/etc/hotplug/input.agent hotplug-2004_03_11/etc/hotplug/input.agent
--- hotplug-2004_03_11.orig/etc/hotplug/input.agent	2004-03-12 08:13:38.000000000 +0900
+++ hotplug-2004_03_11/etc/hotplug/input.agent	2004-03-14 01:56:42.000000000 +0900
@@ -167,16 +167,16 @@
 	product=$(($5))
 	version=$(($6))
 
-	evBits=$(($7))
-	keyBits=$(($8))
-	relBits=$(($9))
+	evBits=$((0x$7))
+	keyBits="$8"
+	relBits=$((0x$9))
 
 	shift 9
-	absBits=$(($1))
-	cbsBits=$(($2))
-	ledBits=$(($3))
-	sndBits=$(($4))
-	ffBits=$(($5))
+	absBits="$1"
+	mscBits=$((0x$2))
+	ledBits="$3"
+	sndBits=$((0x$4))
+	ffBits="$5"
 	driverInfo=$(($6))
 
 	: checkmatch $module

Anyway, I wonder all these bits are parsed as *Bits="$<n>" and use
input_match_bits.  Why only ledBits that is only 1 unsigned int value 
uses input_match_bits?

Thanks,
Fumitoshi UKAI


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&opÌk
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel