質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Linux

Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

1回答

1615閲覧

USBのバッファーについて質問します。

nqf27650

総合スコア115

Linux

Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

0クリップ

投稿2017/03/11 11:09

bufは、unsigned charと宣言されている為に、strcpyとstrlenの
2個のコードでは、エラー: invalid conversion from 'unsigned char*' to 'char*' [-fpermissive]となります。

charとすれば、このエラーはなくなります。
しかし、libusb_bulk_transfer(...buf....)の方でエラーになります。

Q1)旨い解決方法はありませんか?

============================ ^
// bulk send to EP1
unsigned char buf[MAXTRANS];
strcpy(buf,"abcdef");
size = strlen(buf);
ret=libusb_bulk_transfer(h_dev,0x01,buf,size,&transferred,1000);

以上宜しくお願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

こんにちは。

意味的には、charは文字列で、unsigned charはバイト列です。
通常、バイト列に対して文字列用のstrcpy()系の関数を使うと何か問題が起きます。
提示されているソースでは、文字列最後のNULLをコピーするため、文字列の文字数がMAXTRANSと等しい時、不正アクセスするため何が起こるか分かりません。

対策の方針は2つ考えられます。

  1. USB通信は文字列だけの場合は、全体的に文字列主で処理するのが良いです。
  2. USB通信は基本的にバイナリ値をやり取りする。その一部として文字列を送ることがある場合は、全体的にバイト列で処理するのが良いです。

前者の方が安易に対応できます。
後者はそれなりに面倒ですが、本来の姿は後者と思います。

どちらでしょうか?

投稿2017/03/11 11:21

Chironian

総合スコア23272

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

nqf27650

2017/03/11 11:35

毎度お世話になります。 >2) USB通信は基本的にバイナリ値をやり取りする。その一部として文字列を送ることがある場合は、全体的にバイト列で処理するのが良いです。 <ーーこの例では文字列を扱っていますが、本番のAppでは、バイト列を処理しなければなりません。 0x12345678という32ビットとか又はバイトデーターの8ビットの送受信を行います。 そこで、参考の為に、strcpy(buf,"abcdef");とかsize = strlen(buf); で バイト列を処理出来るでしょうか? Qtでは、コンパイルエラーになるようです。 お手数でなければ、ご回答ください。
nqf27650

2017/03/11 12:29

毎度お世話になります。 その後、貴方の回答を元に考えて一つの回答をえました。 つまり、1バイトずつ、forループでcopyすれば良いようです。 親切な回答有難う御座いました。 注)サイトから適当なコードをdownLoadし学習しています。
Chironian

2017/03/11 13:32

バイト列ならばlibusb_bulk_transfer()側はそのままで、データを組み立てる方で対処するべきですから、正しい対処と思います。 ちなみに、1バイト毎のコピーでもOKですが、memcpy()を使うのも手です。こちらはvoid*なのでunsigned char*も受け取れます。 C++らしく、std::copy( https://cpprefjp.github.io/reference/algorithm/copy.html )と言う手もあります。イテレータを知らないとちょっと大変ですので、時間を取れる時にトライされた方が良いかも。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問