2008/8/8 FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>:
> On Sat, 2 Aug 2008 17:19:18 +0200
> "Adel Gadllah" <adel.gadllah@xxxxxxxxx> wrote:
>
>> vs fix error handling and compiler warnings
>>
>> ----------------
>>
>> This patch changes the interface of the cmd filter to use a +/- notation like
>> echo -- +0x02 +0x03 -0x08 .
>> If neither + or - is given it defaults to + (allow command).
>>
>> Note: The interface was added in 2.6.17-rc1 and is unused
>> and undocumented so far so it's safe to change it.
>>
>> Cc: matthew@xxxxxx
>> Cc: fujita.tomonori@xxxxxxxxxxxxx,
>> Cc: jens.axboe@xxxxxxxxxx
>> Cc: James.Bottomley@xxxxxxxxxxxxxxxxxxxxx
>> Cc: dan.j.williams@xxxxxxxxx
>> Cc: pjones@xxxxxxxxxx
>> Cc: viro@xxxxxxxxxxxxxxxxxx
>> Cc: dougg@xxxxxxxxxx
>> Signed-off-by: Adel Gadllah <adel.gadllah@xxxxxxxxx>
>>
>> block/cmd-filter.c | 44 +++++++++++++++++++++++++++-----------------
>> 1 files changed, 27 insertions(+), 17 deletions(-)
>
> The patch is corrupted?
>
> fujita@viola:~/git/linux-2.6$ patch -p1 < /home/fujita/Mail/kernel/scsi/1494
> patching file block/cmd-filter.c
> patch: **** malformed patch at line 92: blk_scsi_cmd_filter *filter, char *page,
argh... stupid gmail will resend once I am home. (ie. later today)
>> diff --git a/block/cmd-filter.c b/block/cmd-filter.c
>> index eec4404..cd33f7f 100644
>> --- a/block/cmd-filter.c
>> +++ b/block/cmd-filter.c
>> @@ -84,8 +84,8 @@ static ssize_t rcf_cmds_show(struct
>> blk_scsi_cmd_filter *filter, char *page,
>>
>> for (i = 0; i < BLK_SCSI_MAX_CMDS; i++) {
>> if (test_bit(i, okbits)) {
>> - sprintf(npage, "%02x", i);
>> - npage += 2;
>> + sprintf(npage, "0x%02x", i);
>> + npage += 4;
>> if (i < BLK_SCSI_MAX_CMDS - 1)
>> sprintf(npage++, " ");
>> }
>> @@ -111,32 +111,42 @@ static ssize_t rcf_writecmds_show(struct
>> blk_scsi_cmd_filter *filter,
>> static ssize_t rcf_cmds_store(struct blk_scsi_cmd_filter *filter,
>> const char *page, size_t count, int rw)
>> {
>> - ssize_t ret = 0;
>> unsigned long okbits[BLK_SCSI_CMD_PER_LONG], *target_okbits;
>> - int cmd, status, len;
>> + int cmd, status, set;
>> substring_t ss;
>> + char *p;
>>
>> - memset(&okbits, 0, sizeof(okbits));
>> + if (rw == READ) {
>> + memcpy(&okbits, filter->read_ok, sizeof(okbits));
>> + target_okbits = filter->read_ok;
>> + } else {
>> + memcpy(&okbits, filter->write_ok, sizeof(okbits));
>> + target_okbits = filter->write_ok;
>> + }
>> +
>> + while ((p = strsep((char **)&page, " ")) != NULL) {
>> + set = 1;
>> +
>> + if (p[0] == '-') {
>> + set = 0;
>> + p++;
>> + }
>>
>> - for (len = strlen(page); len > 0; len -= 3) {
>> - if (len < 2)
>> - break;
>> - ss.from = (char *) page + ret;
>> - ss.to = (char *) page + ret + 2;
>> - ret += 3;
>> + if (p[0] == '+')
>> + p++;
>> + ss.from = (char *) p;
>> + ss.to = (char *) p + 4;
>> status = match_hex(&ss, &cmd);
>
> how about using simple_strtol here? It can handle an value too.
by value you mean a non hex input?
> strchr(str, ' ') would make the intention clear here rather than using
> '4' ?
yeah will do that
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
