xmegaのstrstr関数で文字検索

質問

現在ATMEL社のATxmega128A1というCPUでプログラミングを行っております。

テンキーパネルでのコマンド操作機能というのを設置しているのですが、テンキー操作で、”＃”ボタンが押されると

u8 key_proc(u8 * buf,int length)

この関数が必ず呼ばれます。

bufという引数に”＃”ボタンが押されるまでに押されたボタンのキャラクタデータが収納されたSRAMの番地が格納されています。

例えば、端末の設定などを変更したい場合に使用する4文字のキーがEEPROMに格納されていて、それを変更したい場合のコマンドを追加したいと思っています。

そのEEPROMに格納されている4文字キーというのは

(u16)gEnv.main_set.key;

というEEPROMに”1234”というデータが入っているのですが、

bufというSRAMのメモリに例えば”1234Z4321#”

このようなコマンドが来たらもともと”1234”という現在のkeyと合致して、なおかつ次の文字が”Z”という文字が来たら、次の”4321”というデータに(u16)gEnv.main_set.key;のEEPROMの内容を変更するというプログラムが作りたいと思っています。

現在このようなプログラムでこの機能を実現しています。

#if 1
pass16 = (u16)gEnv.main_set.key;
sprintf((char *)str,"%04d",pass16);
str[4] = 0;

memcpy(value,buf,4);

if(strstr((char *)value,(char *)str) != 0)
{
if(buf[4] == 'Z' && buf[9] == '#')
{

value[0] = buf[5];
value[1] = buf[6];
value[2] = buf[7];
value[3] = buf[8];

gEnv.main_set.key = strtoul(value,NULL,10);
env_save(&gEnv);
return true;
}
}
#endif

このプログラムを作ったことで疑問に思っていることなのですが、

buf[4] == 'Z' であるかどうかをstrstrとは別のコードで判定しているのですが、

sprintf((char *)str,"%04dZ",pass16);

このようなコードで"1234Z"という文字列をstrstr検索して判定することはできないのでしょうか?

現在”sprintf((char *)str,"%04dZ",pass16);”としてしまうと、プログラムが暴走したような状態になり、正常な動作ができなくなってしまう現象が出てしまいます。

どうぞ、ご教示よろしくお願い致します。

回答No.4

chie65535 · ベストアンサー

＞sprintf((char *)str,"%04dZ",pass16);
＞この記述が1行入っただけで暴走してしまうのか大変不思議に思っております。

sprintfの書式指定の"Z"が「長さ修飾子」として機能していると思われます。

sprintf((char *)str,"%04d\x5a",pass16);

として下さい。

なお、printf系関数の「書式を書く部分」に

sprintf((char *)str,"%04dM1#",pass16);
sprintf((char *)str,"%04dZ",pass16);

のように「特定の文字、文字列を指定する」のは「書式の修飾子として判断される危険があり、意図した結果にならない場合がある」ので、事故の元です。

こういう「思わぬ事故」を防ぐ為、多少、冗長になってしまうのを承知の上で

sprintf((char *)str,"%04d%s",pass16,"M1#");
sprintf((char *)str,"%04d%s",pass16,"Z");

と書くようにしましょう。

お礼

回答頂きありがとうございます。
解決しました。助かります。

回答No.3

wormhole

>memcpy(value,buf,4);
>if(strstr((char *)value,(char *)str) != 0)

strの検索対象であるvalueは検索直前にbufから4バイトコピーしていますが、
valueはNUL文字分含めた最低5バイト分ありますか？
またvalue[4]はNUL文字ですか？("Z”を含めないで検索する場合の話です)

あと、strstrではなくstrcmpでいいような。
さらにはvalueにコピーして検索せずにmemcmpでいいような

pass16 = (u16)gEnv.main_set.key;
sprintf((char *)str,"%04dZ",pass16);
// str[4] = 0; 元々これは不要

if(memcmp(buf, str, 5) == 0 && buf[9] == '#')
{
memcpy(value, &buf[5], 4);
value[4] = '\0';
gEnv.main_set.key = strtoul(value,NULL,10);
env_save(&gEnv);
return true;
}

お礼

回答頂きありがとうございます。

サンプルコード頂きありがとうございます。助かります。

#if 1
pass16 = (u16)gEnv.main_set.key;
sprintf((char *)str,"%04dZ",pass16);

if(memcmp(buf,str,5)==0 && buf[9]=='#')
{
memcpy(value,&buf[5],4);
value[4] = '\0';

gEnv.main_set.key= strtoul(value,NULL,10);
env_save(&gEnv);
return true;
}

#endif

このコードに差し替えてコンパイルして実行してみたのですが、
前回と同様の暴走したような動作をしてしまいます。

結局、次のソースコードに戻して動作させています。

なお、memcpy(value,buf,4);でブレイクをかけて
value[4]の値を調べたところ"0x82"となっていました。

#if 1
pass16 = (u16)gEnv.main_set.key;
sprintf((char *)str,"%04d",pass16);
//sprintf((char *)str,"%04dZ",pass16);
//str[5] = 0;

memcpy(value,buf,4);

if(strstr((char *)value,(char *)str) != 0)
{
if(buf[4] == 'Z' && buf[9] == '#')
{

value[0] = buf[5];
value[1] = buf[6];
value[2] = buf[7];
value[3] = buf[8];

gEnv.main_set.key= strtoul(value,NULL,10);
env_save(&gEnv);
return true;
}
}
#endif

回答No.2

chie65535

＞プログラムが暴走したような状態になり

char str[5];
とかって定義になっていれば「4文字＋EOS」の分しかメモリが無いので、5文字以上の文字列をstrの場所に書き込むと暴走します。

お礼

回答頂きありがとうございます。

もう一度、今回の関数を見直してみました。
strの配列が少なくて今回の現象が起きているようではないようです。

この"key_proc"関数内には

sprintf((char *)str,"%04dM1#",pass16);
if(strstr((char *)buf,(char *)str) != 0)

このようにして判定をしている箇所もあるのですが、これでは暴走するような現象は起きていないのに、

sprintf((char *)str,"%04dZ",pass16);

この記述が1行入っただけで暴走してしまうのか大変不思議に思っております。

u8 key_proc(u8 * buf,int length)
{

----------(中略)----------------------------------------

u8 str[11];

----------(中略)----------------------------------------

pass16 = (u16)gEnv.main_set.service_code;
sprintf((char *)str,"%04dM1#",pass16);
str[7] =0;
if(strstr((char *)buf,(char *)str) != 0)
{
devicePairingNumber = OPER_PAIRING_REMO_DRIVER;
oper_insert(OPER_PAIRING_REMO_TESTOPER,1);
return true;
}

----------(中略)----------------------------------------

}

回答No.1

wormhole

strの型はどうなってるんですか？

お礼

回答頂きありがとうございます。

strの型ですが、

u8 str[11];

このように宣言しています。