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

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

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

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Q&A

1回答

1300閲覧

Javaの静的解析の指摘について(overflow)

izuki_y

総合スコア65

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

0グッド

1クリップ

投稿2017/02/06 11:32

いつもお世話になっております。
Javaの静的解析による指摘事項で質問があります。

Javaコードで静的解析を行った所以下の様な指摘を受けました。

Unintentional integer overflow

Potentially overflowing expression b2u(mRecvBuf[18]) << 8 with type int(32 bit, signed) is
evaluated using 32-bit arithmetic, and then used in a context that expects an expression of type long(64 bits, signed)

4つのbyte値を結合してlongに格納するコードで発生しました。
以下の通りです。

java

1public class Test{ 2 private static byte[] mRecvBuf = new byte[500]; 3 public static void main(String []args){ 4 5 mRecvBuf[16] = (byte) 0x66; 6 mRecvBuf[17] = (byte) 0x77; 7 mRecvBuf[18] = (byte) 0x88; 8 mRecvBuf[19] = (byte) 0x99; 9 10 // こっちでやると上記の指摘が発生する 11 long lRet = (long)( b2u(mRecvBuf[16]) << 3 * 8) + 12 (long)( b2u(mRecvBuf[17]) << 2 * 8) + 13 (long)( b2u(mRecvBuf[18]) << 1 * 8) + 14 (long)( b2u(mRecvBuf[19])); 15 16 System.out.println("lRet( " + lRet + "=" + Long.toHexString(lRet)); 17 18 // こっちだとなぜか指摘受けない 19 int iRet =( b2u(mRecvBuf[16]) << 3 * 8) + 20 ( b2u(mRecvBuf[17]) << 2 * 8) + 21 ( b2u(mRecvBuf[18]) << 1 * 8) + 22 ( b2u(mRecvBuf[19])); 23 24 System.out.println("iRet( " + iRet + "=" + Integer.toHexString(iRet)); 25 } 26 27 public static int b2u(byte b) { 28 return b & 255; 29 } 30}

桁あふれが問題なのは分かるけど符号が影響しなくて良いようにlongを使用しましたが int だと指摘が出ずに long だと指摘を受けます。

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

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

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

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

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

guest

回答1

0

ビットシフトしてからlongキャストしてるのが意味がないと言うことでは?
intに切り詰めてからキャストされるため、上位32ビットが0になり無意味ですよ。
もとよりintとして扱ってるなら問題ないと判定されているのでは。

投稿2017/02/06 11:45

swordone

総合スコア20649

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問