https://www.opennet.ru/openforum/vsluhforumID9/9136.html Всем доброго времени суток! <br>Заставили тут меня разбираться с драйверами под линукс( Но опыт написания этих драйверов напрочь отсутствует( <br>Перейду сразу к вопросам.<br>Есть функция <br>static ssize_t hmc5843_store_rate(struct device *dev,<br>+ struct device_attribute *attr,<br>+ const char *buf, size_t count)<br>+{<br>+struct i2c_client *client = to_i2c_client(dev);<br>+struct hmc5843 *hmc5843 = i2c_get_clientdata(client);<br>+unsigned long val;<br>+int status = count;<br>+<br>+if ((strict_strtol(buf, 10, &val) &lt; 0) &#124;&#124; (val &gt; 6))<br>+return -EINVAL;<br>+mutex_lock(&hmc5843-&gt;lock);<br>+if (hmc5843-&gt;rate != val) {<br>+hmc5843-&gt;rate = val;<br>+hmc5843-&gt;ipdev-&gt;poll_interval = hmc5843_sample_interval[val];<br>+status = hmc5843_write_register(hmc5843, HMC5843_CFG_A_REG);<br>+}<br>+mutex_unlock(&hmc5843-&gt;lock);<br>+return status;<br>+}<br><br>Исходник находится по ссылке https://patchwork.kernel.org/patch/66557/<br>Так вот. <br>Вот объявлена переменная <br>unsigned long val; <br>и сразу же через пару строк идет проверка<br>if (hmc https://www.opennet.ru/openforum/vsluhforumID9/9136.html#4 Thu, 02 Jun 2011 11:18:20 GMT &gt; Спасибо всем заранее.<br><br>1. unsigned long val; должно быть long val;<br><br>2. Перед использованием можно инициализировать - long val = 7; <br><br>3. strict_strtol(buf, 10, &val) возвращает 0 в случае успеха, остальные значения - ошибка.<br><br>4. int status должно быть ssize_t, т.к. вся функция возвращает ssize_t (на 32бита пофиг, на 64 это long int) <br><br>---<br> long val;<br> ssize_t status = count;<br><br> val = 7;<br> if ((strict_strtol(buf, 10, &val) != 0) &#124;&#124; (val &gt; 6))<br> return -EINVAL;<br><br><br>P.S. <br>Оно уже давно в ядре :)<br>http://lxr.linux.no/#linux+v2.6.39/drivers/staging/iio/magnetometer/hmc5843.c<br><br><br> https://www.opennet.ru/openforum/vsluhforumID9/9136.html#3 Mon, 30 May 2011 02:52:38 GMT &gt;&gt; + if ((strict_strtol(buf, 10, &val) &lt; 0) &#124;&#124; (val <br>&gt;&gt; &gt; 6)) <br>&gt; но вообще опасно. даже если стандарт скажет, что функция таки вызовется <br>&gt; до , всегда <br>&gt; есть возможность нарваться на хитрооптимизирующий компилятор.<br>&gt; я бы разнес на 2 строки ..<br><br>Такое сплошь и рядом встречается и пока мир не обрушился. Так что все в порядке.<br> https://www.opennet.ru/openforum/vsluhforumID9/9136.html#2 Mon, 30 May 2011 01:26:54 GMT &gt; + if ((strict_strtol(buf, 10, &val) &lt; 0) &#124;&#124; (val <br>&gt; &gt; 6)) <br><br> но вообще опасно. даже если стандарт скажет, что функция таки вызовется до , всегда<br>есть возможность нарваться на хитрооптимизирующий компилятор.<br><br> я бы разнес на 2 строки .. <br> https://www.opennet.ru/openforum/vsluhforumID9/9136.html#1 Wed, 18 May 2011 20:55:27 GMT + if ((strict_strtol(buf, 10, &val) &lt; 0) &#124;&#124; (val &gt; 6))<br><br>