forked from wooyunwang/Fortify
-
Notifications
You must be signed in to change notification settings - Fork 0
/
不检测返回值
24 lines (24 loc) · 994 Bytes
/
不检测返回值
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<pre>
FileInputStream fis;
byte[] byteArray = new byte[1024];
for (Iterator i=users.iterator();
i.hasNext();) {
String userName = (String) i.next();
String pFileName = PFILE_ROOT + "/"
+ userName;
fis = new FileInputStream(pFileName);
int bRead = 0;
while (bRead <
1024) {
int rd = fis.read(byteArray, bRead, 1024 - bRead);
if (rd == -1) {
throw new IOException("file is unusually small");
}
bRead += rd;
}
// could add check to see if file is too large here
fis.close();
processPFile(userName, byteArray);
}
</pre>
注:因为该问题的修复相当地复杂,您可能试图使用一个更简单的方法,例如在开始阅读前检查文件的大小。这种方法将导致应用程序容易受到文件系统 race condition 的攻击,凭借这个攻击者可以在文件大小检查和从文件调用读取数据之间使用恶意文件替换结构良好的文件。