Skip to content

Commit 5fba508

Browse files
committed
HDFS-1153. dfsnodelist.jsp should handle invalid input parameters. Contributed by Ravi Phulari
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1339706 13f79535-47bb-0310-9956-ffa450edef68
1 parent ab69ce9 commit 5fba508

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,9 @@ Release 2.0.1-alpha - UNRELEASED
211211
HDFS-3433. GetImageServlet should allow administrative requestors when
212212
security is enabled. (atm)
213213

214+
HDFS-1153. dfsnodelist.jsp should handle invalid input parameters.
215+
(Ravi Phulari via eli)
216+
214217
Release 2.0.0-alpha - UNRELEASED
215218

216219
INCOMPATIBLE CHANGES

hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NamenodeJspHelper.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -587,6 +587,10 @@ void generateNodesList(ServletContext context, JspWriter out,
587587

588588
whatNodes = request.getParameter("whatNodes"); // show only live or only
589589
// dead nodes
590+
if (null == whatNodes || whatNodes.isEmpty()) {
591+
out.print("Invalid input");
592+
return;
593+
}
590594
sorterField = request.getParameter("sorter/field");
591595
sorterOrder = request.getParameter("sorter/order");
592596
if (sorterField == null)
@@ -714,6 +718,8 @@ void generateNodesList(ServletContext context, JspWriter out,
714718
}
715719
out.print("</table>\n");
716720
}
721+
} else {
722+
out.print("Invalid input");
717723
}
718724
out.print("</div>");
719725
}

0 commit comments

Comments
 (0)