public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
* [SCM]  master: Fix #5383. Peek without arguments causes NPE.
@ 2007-11-21 10:03 pmuldoon
  0 siblings, 0 replies; only message in thread
From: pmuldoon @ 2007-11-21 10:03 UTC (permalink / raw)
  To: frysk-cvs

The branch, master has been updated
       via  f8936dca409520dc5833354037912026ab2a9a7b (commit)
      from  c6b2144b5562906445054a6fb3d991afa4356a7c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit f8936dca409520dc5833354037912026ab2a9a7b
Author: Phil Muldoon <pmuldoon@redhat.com>
Date:   Wed Nov 21 10:02:54 2007 +0000

    Fix #5383. Peek without arguments causes NPE.
    
    2007-11-21  Phil Muldoon  <pmuldoon@redhat.com>
    
    	* PeekCommand.java (interpret): Check that arguments are > 0.

-----------------------------------------------------------------------

Summary of changes:
 frysk-core/frysk/hpd/ChangeLog        |    4 ++++
 frysk-core/frysk/hpd/PeekCommand.java |    3 +++
 2 files changed, 7 insertions(+), 0 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/hpd/ChangeLog b/frysk-core/frysk/hpd/ChangeLog
index 26827e1..56c1b6c 100644
--- a/frysk-core/frysk/hpd/ChangeLog
+++ b/frysk-core/frysk/hpd/ChangeLog
@@ -1,3 +1,7 @@
+2007-11-21  Phil Muldoon  <pmuldoon@redhat.com>
+
+	* PeekCommand.java (interpret): Check that arguments are > 0.
+	
 2007-11-20  Rick Moseley  <rmoseley@redhat.com>
 
 	* PeekCommand.java: Fix bz# 5380(make peek command
diff --git a/frysk-core/frysk/hpd/PeekCommand.java b/frysk-core/frysk/hpd/PeekCommand.java
index 22d8db4..9c9ec52 100644
--- a/frysk-core/frysk/hpd/PeekCommand.java
+++ b/frysk-core/frysk/hpd/PeekCommand.java
@@ -58,6 +58,9 @@ public class PeekCommand extends ParameterizedCommand {
     }
 
     void interpret(CLI cli, Input cmd, Object options) {
+        if (cmd.size() < 1)
+	    throw new InvalidCommandException("Not enough "
+                + "parameters. Please specify an addess to peek at.");
 	if (cmd.size() > 1) {
 	    throw new InvalidCommandException("Too many parameters.");
 	}


hooks/post-receive
--
frysk system monitor/debugger


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-11-21 10:03 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-11-21 10:03 [SCM] master: Fix #5383. Peek without arguments causes NPE pmuldoon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).