|
 |
d5c145 |
From 56134ff5442bee4e128b189bb86cfc97dcb6f60a Mon Sep 17 00:00:00 2001
|
|
|
d5c145 |
From: Petr Machata <pmachata@redhat.com>
|
|
|
d5c145 |
Date: Fri, 10 Jan 2014 20:05:15 +0100
|
|
|
d5c145 |
Subject: [PATCH 1/2] Add a new per-breakpoint callback on_install
|
|
|
d5c145 |
|
|
|
d5c145 |
---
|
|
|
d5c145 |
breakpoint.h | 9 ++++++++-
|
|
|
d5c145 |
breakpoints.c | 11 ++++++++++-
|
|
|
d5c145 |
2 files changed, 18 insertions(+), 2 deletions(-)
|
|
|
d5c145 |
|
|
|
d5c145 |
diff --git a/breakpoint.h b/breakpoint.h
|
|
|
d5c145 |
index 95964a8..c36f673 100644
|
|
|
d5c145 |
--- a/breakpoint.h
|
|
|
d5c145 |
+++ b/breakpoint.h
|
|
|
d5c145 |
@@ -1,6 +1,6 @@
|
|
|
d5c145 |
/*
|
|
|
d5c145 |
* This file is part of ltrace.
|
|
|
d5c145 |
- * Copyright (C) 2012, 2013 Petr Machata, Red Hat Inc.
|
|
|
d5c145 |
+ * Copyright (C) 2012,2013,2014 Petr Machata, Red Hat Inc.
|
|
|
d5c145 |
* Copyright (C) 2009 Juan Cespedes
|
|
|
d5c145 |
*
|
|
|
d5c145 |
* This program is free software; you can redistribute it and/or
|
|
|
d5c145 |
@@ -46,6 +46,7 @@
|
|
|
d5c145 |
struct bp_callbacks {
|
|
|
d5c145 |
void (*on_hit)(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
void (*on_continue)(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
+ void (*on_install)(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
void (*on_retract)(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
|
|
|
d5c145 |
/* Create a new breakpoint that should handle return from the
|
|
|
d5c145 |
@@ -84,6 +85,12 @@ void breakpoint_on_continue(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
* the instruction underneath it). */
|
|
|
d5c145 |
void breakpoint_on_retract(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
|
|
|
d5c145 |
+/* Call ON_INSTALL handler of BP, if any is set. This should be
|
|
|
d5c145 |
+ * called after the breakpoint is enabled for the first time, not
|
|
|
d5c145 |
+ * every time it's enabled (such as after stepping over a site of a
|
|
|
d5c145 |
+ * temporarily disabled breakpoint). */
|
|
|
d5c145 |
+void breakpoint_on_install(struct breakpoint *bp, struct process *proc);
|
|
|
d5c145 |
+
|
|
|
d5c145 |
/* Call GET_RETURN_BP handler of BP, if any is set. If none is set,
|
|
|
d5c145 |
* call CREATE_DEFAULT_RETURN_BP to obtain one. */
|
|
|
d5c145 |
int breakpoint_get_return_bp(struct breakpoint **ret,
|
|
|
d5c145 |
diff --git a/breakpoints.c b/breakpoints.c
|
|
|
d5c145 |
index 947cb71..c3fa275 100644
|
|
|
d5c145 |
--- a/breakpoints.c
|
|
|
d5c145 |
+++ b/breakpoints.c
|
|
|
d5c145 |
@@ -1,6 +1,6 @@
|
|
|
d5c145 |
/*
|
|
|
d5c145 |
* This file is part of ltrace.
|
|
|
d5c145 |
- * Copyright (C) 2006,2007,2011,2012,2013 Petr Machata, Red Hat Inc.
|
|
|
d5c145 |
+ * Copyright (C) 2006,2007,2011,2012,2013,2014 Petr Machata, Red Hat Inc.
|
|
|
d5c145 |
* Copyright (C) 2009 Juan Cespedes
|
|
|
d5c145 |
* Copyright (C) 1998,2001,2002,2003,2007,2008,2009 Juan Cespedes
|
|
|
d5c145 |
* Copyright (C) 2006 Ian Wienand
|
|
|
d5c145 |
@@ -85,6 +85,14 @@ breakpoint_on_retract(struct breakpoint *bp, struct process *proc)
|
|
|
d5c145 |
(bp->cbs->on_retract)(bp, proc);
|
|
|
d5c145 |
}
|
|
|
d5c145 |
|
|
|
d5c145 |
+void
|
|
|
d5c145 |
+breakpoint_on_install(struct breakpoint *bp, struct process *proc)
|
|
|
d5c145 |
+{
|
|
|
d5c145 |
+ assert(bp != NULL);
|
|
|
d5c145 |
+ if (bp->cbs != NULL && bp->cbs->on_install != NULL)
|
|
|
d5c145 |
+ (bp->cbs->on_install)(bp, proc);
|
|
|
d5c145 |
+}
|
|
|
d5c145 |
+
|
|
|
d5c145 |
int
|
|
|
d5c145 |
breakpoint_get_return_bp(struct breakpoint **ret,
|
|
|
d5c145 |
struct breakpoint *bp, struct process *proc)
|
|
|
d5c145 |
@@ -229,6 +237,7 @@ breakpoint_turn_on(struct breakpoint *bp, struct process *proc)
|
|
|
d5c145 |
if (bp->enabled == 1) {
|
|
|
d5c145 |
assert(proc->pid != 0);
|
|
|
d5c145 |
enable_breakpoint(proc, bp);
|
|
|
d5c145 |
+ breakpoint_on_install(bp, proc);
|
|
|
d5c145 |
}
|
|
|
d5c145 |
return 0;
|
|
|
d5c145 |
}
|
|
|
d5c145 |
--
|
|
|
d5c145 |
1.7.6.5
|
|
|
d5c145 |
|