java/code/src/com/redhat/rhn/frontend/action/rhnpackage/PackageDetailsAction.java | 11
++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
New commits:
commit 3022ae5840f925a031c0200fb9820f3cc9004027
Author: Pradeep Kilambi <pkilambi(a)redhat.com>
Date: Mon Aug 3 13:08:04 2009 -0400
515219 - We can have a channel with null description. In the packagedetailsaction we
call replace on description without checking if its a valid string resulting in Null
Pointer Exception.
diff --git
a/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/PackageDetailsAction.java
b/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/PackageDetailsAction.java
index 078fcb2..9bd7cb0 100644
--- a/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/PackageDetailsAction.java
+++ b/java/code/src/com/redhat/rhn/frontend/action/rhnpackage/PackageDetailsAction.java
@@ -100,8 +100,15 @@ public class PackageDetailsAction extends RhnAction {
}
request.setAttribute("pack", pkg);
- request.setAttribute("description",
- pkg.getDescription().replace("\n",
"<BR>\n"));
+ // description can be null.
+ if (pkg.getDescription() != null) {
+ request.setAttribute("description",
+ pkg.getDescription().replace("\n",
"<BR>\n"));
+ }
+ else {
+ request.setAttribute("description",
+ pkg.getDescription());
+ }
request.setAttribute("packArches",
PackageFactory.findPackagesWithDifferentArch(pkg));
request.setAttribute("pid", pid);
Show replies by date