Vojtech Szocs has uploaded a new change for review. Change subject: engine: Fix potential NumberFormatException in OsRepositoryImpl ......................................................................
engine: Fix potential NumberFormatException in OsRepositoryImpl Added NumberFormatException check in OsRepositoryImpl.getInt method. Change-Id: I3ec4c9704d35b7f2955ed41761d4f019969897c1 Signed-off-by: Vojtech Szocs <vsz...@redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImpl.java 1 file changed, 5 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/17/17117/1 diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImpl.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImpl.java index c401d44..fcbe92d 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImpl.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImpl.java @@ -186,7 +186,11 @@ } private int getInt(String value, int defaultValue) { - return value == null ? defaultValue : Integer.parseInt(value); + try { + return value == null ? defaultValue : Integer.parseInt(value); + } catch (NumberFormatException ex) { + return defaultValue; + } } /** -- To view, visit http://gerrit.ovirt.org/17117 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3ec4c9704d35b7f2955ed41761d4f019969897c1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vsz...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches