backport bug [ 1192315 ] 'clear -1' in pdb

2 files changed, 6 insertions, 1 deletions

diff --git a/Lib/pdb.py b/Lib/pdb.py
index 7b5dffa..b00f68b 100755
--- a/Lib/pdb.py
+++ b/Lib/pdb.py
@@ -450,11 +450,14 @@ class Pdb(bdb.Bdb, cmd.Cmd):
             return
         numberlist = arg.split()
         for i in numberlist:
+            if not (0 <= i < len(bdb.Breakpoint.bpbynumber)):
+                print 'No breakpoint numbered', i
+                continue
             err = self.clear_bpbynumber(i)
             if err:
                 print '***', err
             else:
-                print 'Deleted breakpoint %s ' % (i,)
+                print 'Deleted breakpoint', i
     do_cl = do_clear # 'c' is already an abbreviation for 'continue'
 
     def do_where(self, arg):
diff --git a/Misc/NEWS b/Misc/NEWS
index 55219f5..48a7d89 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -70,6 +70,8 @@ Library
 
 - The sets module can now properly compute s-=s and s^=s as an empty set.
 
+- Bug #1192315: Disallow negative arguments to clear() in pdb.
+
 - Patch #827386: Support absolute source paths in msvccompiler.py.
 
 - Fix a problem in Tkinter introduced by SF patch #869468: delete bogus