diff options
| author | Abhishek Koneru <akoneru@redhat.com> | 2012-06-06 16:55:54 -0400 |
|---|---|---|
| committer | Endi Sukma Dewata <edewata@redhat.com> | 2012-06-14 17:14:19 -0500 |
| commit | c53ca291e21761f1de5417ef596afba395a7f5d1 (patch) | |
| tree | 47a0cd8ecd9d36d414d9230282704e9f784d0a71 /base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java | |
| parent | 084a8cd360c7508febde06415d727d7d247b16ad (diff) | |
| download | pki-c53ca291e21761f1de5417ef596afba395a7f5d1.tar.gz pki-c53ca291e21761f1de5417ef596afba395a7f5d1.tar.xz pki-c53ca291e21761f1de5417ef596afba395a7f5d1.zip | |
Fixes for NULL_RETURNS Coverity Issues - Part 2
Diffstat (limited to 'base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java')
| -rwxr-xr-x | base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java b/base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java index 0e3325005..f4ea79c2b 100755 --- a/base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java +++ b/base/silent/src/com/netscape/pkisilent/argparser/ArgParser.java @@ -1610,7 +1610,7 @@ public class ArgParser { return (rec != null) ? rec.valTypeName() : null; } - private Object createResultHolder(Record rec) { + private Object createResultHolder(Record rec) throws ArgParseException { if (rec.numValues == 1) { switch (rec.type) { case Record.LONG: { @@ -1648,7 +1648,9 @@ public class ArgParser { } } } - return null; // can't happen + + throw new ArgParseException("Bad parameters in the Record for Result Holder. Type :" + rec.type + + " ,Number of Values : " + rec.numValues); // can't happen } static void stringToArgs(Vector<String> vec, String s, |