sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1 | |
| 2 | /*--------------------------------------------------------------------*/ |
sewardj | 267100d | 2005-04-24 12:33:12 +0000 | [diff] [blame] | 3 | /*--- Management of error messages. m_errormgr.c ---*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 4 | /*--------------------------------------------------------------------*/ |
| 5 | |
| 6 | /* |
njn | b9c427c | 2004-12-01 14:14:42 +0000 | [diff] [blame] | 7 | This file is part of Valgrind, a dynamic binary instrumentation |
| 8 | framework. |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 9 | |
sewardj | 9eecbbb | 2010-05-03 21:37:12 +0000 | [diff] [blame] | 10 | Copyright (C) 2000-2010 Julian Seward |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 11 | jseward@acm.org |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 12 | |
| 13 | This program is free software; you can redistribute it and/or |
| 14 | modify it under the terms of the GNU General Public License as |
| 15 | published by the Free Software Foundation; either version 2 of the |
| 16 | License, or (at your option) any later version. |
| 17 | |
| 18 | This program is distributed in the hope that it will be useful, but |
| 19 | WITHOUT ANY WARRANTY; without even the implied warranty of |
| 20 | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
| 21 | General Public License for more details. |
| 22 | |
| 23 | You should have received a copy of the GNU General Public License |
| 24 | along with this program; if not, write to the Free Software |
| 25 | Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA |
| 26 | 02111-1307, USA. |
| 27 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 28 | The GNU General Public License is contained in the file COPYING. |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 29 | */ |
| 30 | |
njn | c7561b9 | 2005-06-19 01:24:32 +0000 | [diff] [blame] | 31 | #include "pub_core_basics.h" |
sewardj | 4cfea4f | 2006-10-14 19:26:10 +0000 | [diff] [blame] | 32 | #include "pub_core_vki.h" |
sewardj | 6c591e1 | 2011-04-11 16:17:51 +0000 | [diff] [blame] | 33 | #include "pub_core_libcsetjmp.h" |
njn | 24a6efb | 2005-06-20 03:36:51 +0000 | [diff] [blame] | 34 | #include "pub_core_threadstate.h" // For VG_N_THREADS |
njn | 75b65aa | 2005-06-19 19:25:44 +0000 | [diff] [blame] | 35 | #include "pub_core_debugger.h" |
njn | ea27e46 | 2005-05-31 02:38:09 +0000 | [diff] [blame] | 36 | #include "pub_core_debuginfo.h" |
njn | d2b1711 | 2005-04-19 04:10:25 +0000 | [diff] [blame] | 37 | #include "pub_core_errormgr.h" |
njn | d01fef7 | 2005-03-25 23:35:48 +0000 | [diff] [blame] | 38 | #include "pub_core_execontext.h" |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 39 | #include "pub_core_gdbserver.h" |
njn | 97405b2 | 2005-06-02 03:39:33 +0000 | [diff] [blame] | 40 | #include "pub_core_libcbase.h" |
njn | 132bfcc | 2005-06-04 19:16:06 +0000 | [diff] [blame] | 41 | #include "pub_core_libcassert.h" |
njn | eb8896b | 2005-06-04 20:03:55 +0000 | [diff] [blame] | 42 | #include "pub_core_libcfile.h" |
njn | 36a20fa | 2005-06-03 03:08:39 +0000 | [diff] [blame] | 43 | #include "pub_core_libcprint.h" |
njn | 24a6efb | 2005-06-20 03:36:51 +0000 | [diff] [blame] | 44 | #include "pub_core_libcproc.h" // For VG_(getpid)() |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 45 | #include "pub_core_seqmatch.h" |
njn | af1d7df | 2005-06-11 01:31:52 +0000 | [diff] [blame] | 46 | #include "pub_core_mallocfree.h" |
njn | 2024234 | 2005-05-16 23:31:24 +0000 | [diff] [blame] | 47 | #include "pub_core_options.h" |
njn | d0d7c1f | 2005-06-21 00:33:19 +0000 | [diff] [blame] | 48 | #include "pub_core_stacktrace.h" |
njn | 43b9a8a | 2005-05-10 04:37:01 +0000 | [diff] [blame] | 49 | #include "pub_core_tooliface.h" |
njn | 24a6efb | 2005-06-20 03:36:51 +0000 | [diff] [blame] | 50 | #include "pub_core_translate.h" // for VG_(translate)() |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 51 | #include "pub_core_xarray.h" // VG_(xaprintf) et al |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 52 | |
| 53 | /*------------------------------------------------------------*/ |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 54 | /*--- Globals ---*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 55 | /*------------------------------------------------------------*/ |
| 56 | |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 57 | /* After this many different unsuppressed errors have been observed, |
| 58 | be more conservative about collecting new ones. */ |
sewardj | 1d1a226 | 2005-10-20 01:57:29 +0000 | [diff] [blame] | 59 | #define M_COLLECT_ERRORS_SLOWLY_AFTER 100 |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 60 | |
| 61 | /* After this many different unsuppressed errors have been observed, |
| 62 | stop collecting errors at all, and tell the user their program is |
| 63 | evidently a steaming pile of camel dung. */ |
sewardj | 1d1a226 | 2005-10-20 01:57:29 +0000 | [diff] [blame] | 64 | #define M_COLLECT_NO_ERRORS_AFTER_SHOWN 1000 |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 65 | |
| 66 | /* After this many total errors have been observed, stop collecting |
| 67 | errors at all. Counterpart to M_COLLECT_NO_ERRORS_AFTER_SHOWN. */ |
sewardj | 513646f | 2006-05-12 23:12:30 +0000 | [diff] [blame] | 68 | #define M_COLLECT_NO_ERRORS_AFTER_FOUND 10000000 |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 69 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 70 | /* The list of error contexts found, both suppressed and unsuppressed. |
| 71 | Initially empty, and grows as errors are detected. */ |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 72 | static Error* errors = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 73 | |
| 74 | /* The list of suppression directives, as read from the specified |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 75 | suppressions file. Note that the list gets rearranged as a result |
| 76 | of the searches done by is_suppressible_error(). */ |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 77 | static Supp* suppressions = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 78 | |
| 79 | /* Running count of unsuppressed errors detected. */ |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 80 | static UInt n_errs_found = 0; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 81 | |
| 82 | /* Running count of suppressed errors detected. */ |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 83 | static UInt n_errs_suppressed = 0; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 84 | |
njn | 606a4ae | 2009-08-11 00:52:40 +0000 | [diff] [blame] | 85 | /* Running count of unsuppressed error contexts. */ |
| 86 | static UInt n_err_contexts = 0; |
| 87 | |
| 88 | /* Running count of suppressed error contexts. */ |
| 89 | static UInt n_supp_contexts = 0; |
| 90 | |
| 91 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 92 | /* forwards ... */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 93 | static Supp* is_suppressible_error ( Error* err ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 94 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 95 | static ThreadId last_tid_printed = 1; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 96 | |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 97 | /* Stats: number of searches of the error list initiated. */ |
| 98 | static UWord em_errlist_searches = 0; |
| 99 | |
| 100 | /* Stats: number of comparisons done during error list |
| 101 | searching. */ |
| 102 | static UWord em_errlist_cmps = 0; |
| 103 | |
| 104 | /* Stats: number of searches of the suppression list initiated. */ |
| 105 | static UWord em_supplist_searches = 0; |
| 106 | |
| 107 | /* Stats: number of comparisons done during suppression list |
| 108 | searching. */ |
| 109 | static UWord em_supplist_cmps = 0; |
| 110 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 111 | /*------------------------------------------------------------*/ |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 112 | /*--- Error type ---*/ |
| 113 | /*------------------------------------------------------------*/ |
| 114 | |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 115 | /* Errors. Extensible (via the 'extra' field). Tools can use a normal |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 116 | enum (with element values in the normal range (0..)) for 'ekind'. |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 117 | Functions for getting/setting the tool-relevant fields are in |
njn | c7561b9 | 2005-06-19 01:24:32 +0000 | [diff] [blame] | 118 | include/pub_tool_errormgr.h. |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 119 | |
| 120 | When errors are found and recorded with VG_(maybe_record_error)(), all |
| 121 | the tool must do is pass in the four parameters; core will |
| 122 | allocate/initialise the error record. |
| 123 | */ |
| 124 | struct _Error { |
| 125 | struct _Error* next; |
sewardj | dbada27 | 2005-07-02 21:16:30 +0000 | [diff] [blame] | 126 | // Unique tag. This gives the error a unique identity (handle) by |
| 127 | // which it can be referred to afterwords. Currently only used for |
| 128 | // XML printing. |
| 129 | UInt unique; |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 130 | // NULL if unsuppressed; or ptr to suppression record. |
| 131 | Supp* supp; |
| 132 | Int count; |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 133 | |
| 134 | // The tool-specific part |
sewardj | b8b79ad | 2008-03-03 01:35:41 +0000 | [diff] [blame] | 135 | ThreadId tid; // Initialised by core |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 136 | ExeContext* where; // Initialised by core |
njn | d2b1711 | 2005-04-19 04:10:25 +0000 | [diff] [blame] | 137 | ErrorKind ekind; // Used by ALL. Must be in the range (0..) |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 138 | Addr addr; // Used frequently |
| 139 | Char* string; // Used frequently |
| 140 | void* extra; // For any tool-specific extras |
| 141 | }; |
| 142 | |
sewardj | b8b79ad | 2008-03-03 01:35:41 +0000 | [diff] [blame] | 143 | |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 144 | ExeContext* VG_(get_error_where) ( Error* err ) |
| 145 | { |
| 146 | return err->where; |
| 147 | } |
| 148 | |
| 149 | ErrorKind VG_(get_error_kind) ( Error* err ) |
| 150 | { |
| 151 | return err->ekind; |
| 152 | } |
| 153 | |
| 154 | Addr VG_(get_error_address) ( Error* err ) |
| 155 | { |
| 156 | return err->addr; |
| 157 | } |
| 158 | |
| 159 | Char* VG_(get_error_string) ( Error* err ) |
| 160 | { |
| 161 | return err->string; |
| 162 | } |
| 163 | |
| 164 | void* VG_(get_error_extra) ( Error* err ) |
| 165 | { |
| 166 | return err->extra; |
| 167 | } |
| 168 | |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 169 | UInt VG_(get_n_errs_found)( void ) |
| 170 | { |
| 171 | return n_errs_found; |
| 172 | } |
| 173 | |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 174 | /*------------------------------------------------------------*/ |
| 175 | /*--- Suppression type ---*/ |
| 176 | /*------------------------------------------------------------*/ |
| 177 | |
| 178 | /* Note: it is imperative this doesn't overlap with (0..) at all, as tools |
| 179 | * effectively extend it by defining their own enums in the (0..) range. */ |
| 180 | typedef |
| 181 | enum { |
njn | a86f29e | 2006-12-14 02:55:58 +0000 | [diff] [blame] | 182 | // Nb: thread errors are a relic of the time when Valgrind's core |
| 183 | // could detect them. This example is left commented-out as an |
| 184 | // example should new core errors ever be added. |
| 185 | ThreadSupp = -1, /* Matches ThreadErr */ |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 186 | } |
| 187 | CoreSuppKind; |
| 188 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 189 | /* Max number of callers for context in a suppression. */ |
| 190 | #define VG_MAX_SUPP_CALLERS 24 |
| 191 | |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 192 | /* For each caller specified for a suppression, record the nature of |
| 193 | the caller name. Not of interest to tools. */ |
| 194 | typedef |
| 195 | enum { |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 196 | NoName, /* Error case */ |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 197 | ObjName, /* Name is of an shared object file. */ |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 198 | FunName, /* Name is of a function. */ |
| 199 | DotDotDot /* Frame-level wildcard */ |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 200 | } |
| 201 | SuppLocTy; |
| 202 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 203 | typedef |
| 204 | struct { |
| 205 | SuppLocTy ty; |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 206 | Char* name; /* NULL for NoName and DotDotDot */ |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 207 | } |
| 208 | SuppLoc; |
| 209 | |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 210 | /* Suppressions. Tools can get/set tool-relevant parts with functions |
njn | c7561b9 | 2005-06-19 01:24:32 +0000 | [diff] [blame] | 211 | declared in include/pub_tool_errormgr.h. Extensible via the 'extra' field. |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 212 | Tools can use a normal enum (with element values in the normal range |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 213 | (0..)) for 'skind'. */ |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 214 | struct _Supp { |
| 215 | struct _Supp* next; |
| 216 | Int count; // The number of times this error has been suppressed. |
| 217 | Char* sname; // The name by which the suppression is referred to. |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 218 | |
| 219 | // Length of 'callers' |
| 220 | Int n_callers; |
| 221 | // Array of callers, for matching stack traces. First one (name of fn |
| 222 | // where err occurs) is mandatory; rest are optional. |
| 223 | SuppLoc* callers; |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 224 | |
| 225 | /* The tool-specific part */ |
| 226 | SuppKind skind; // What kind of suppression. Must use the range (0..). |
| 227 | Char* string; // String -- use is optional. NULL by default. |
| 228 | void* extra; // Anything else -- use is optional. NULL by default. |
| 229 | }; |
| 230 | |
| 231 | SuppKind VG_(get_supp_kind) ( Supp* su ) |
| 232 | { |
| 233 | return su->skind; |
| 234 | } |
| 235 | |
| 236 | Char* VG_(get_supp_string) ( Supp* su ) |
| 237 | { |
| 238 | return su->string; |
| 239 | } |
| 240 | |
| 241 | void* VG_(get_supp_extra) ( Supp* su ) |
| 242 | { |
| 243 | return su->extra; |
| 244 | } |
| 245 | |
| 246 | |
| 247 | void VG_(set_supp_kind) ( Supp* su, SuppKind skind ) |
| 248 | { |
| 249 | su->skind = skind; |
| 250 | } |
| 251 | |
| 252 | void VG_(set_supp_string) ( Supp* su, Char* string ) |
| 253 | { |
| 254 | su->string = string; |
| 255 | } |
| 256 | |
| 257 | void VG_(set_supp_extra) ( Supp* su, void* extra ) |
| 258 | { |
| 259 | su->extra = extra; |
| 260 | } |
| 261 | |
| 262 | |
| 263 | /*------------------------------------------------------------*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 264 | /*--- Helper fns ---*/ |
| 265 | /*------------------------------------------------------------*/ |
| 266 | |
njn | 0087c50 | 2005-07-01 04:15:36 +0000 | [diff] [blame] | 267 | // Only show core errors if the tool wants to, we're not running with -q, |
| 268 | // and were not outputting XML. |
| 269 | Bool VG_(showing_core_errors)(void) |
| 270 | { |
| 271 | return VG_(needs).core_errors && VG_(clo_verbosity) >= 1 && !VG_(clo_xml); |
| 272 | } |
| 273 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 274 | /* Compare errors, to detect duplicates. |
| 275 | */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 276 | static Bool eq_Error ( VgRes res, Error* e1, Error* e2 ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 277 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 278 | if (e1->ekind != e2->ekind) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 279 | return False; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 280 | if (!VG_(eq_ExeContext)(res, e1->where, e2->where)) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 281 | return False; |
| 282 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 283 | switch (e1->ekind) { |
njn | a86f29e | 2006-12-14 02:55:58 +0000 | [diff] [blame] | 284 | //(example code, see comment on CoreSuppKind above) |
| 285 | //case ThreadErr: |
| 286 | // vg_assert(VG_(needs).core_errors); |
| 287 | // return <something> |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 288 | default: |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 289 | if (VG_(needs).tool_errors) { |
| 290 | return VG_TDICT_CALL(tool_eq_Error, res, e1, e2); |
| 291 | } else { |
njn | 95ec870 | 2004-11-22 16:46:13 +0000 | [diff] [blame] | 292 | VG_(printf)("\nUnhandled error type: %u. VG_(needs).tool_errors\n" |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 293 | "probably needs to be set.\n", |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 294 | e1->ekind); |
njn | 6799325 | 2004-11-22 18:02:32 +0000 | [diff] [blame] | 295 | VG_(tool_panic)("unhandled error type"); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 296 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 297 | } |
| 298 | } |
| 299 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 300 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 301 | /* Helper functions for suppression generation: print a single line of |
| 302 | a suppression pseudo-stack-trace, either in XML or text mode. It's |
| 303 | important that the behaviour of these two functions exactly |
| 304 | corresponds. |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 305 | */ |
| 306 | #define ERRTXT_LEN 4096 |
| 307 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 308 | static void printSuppForIp_XML(UInt n, Addr ip, void* uu_opaque) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 309 | { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 310 | static UChar buf[ERRTXT_LEN]; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 311 | if ( VG_(get_fnname_no_cxx_demangle) (ip, buf, ERRTXT_LEN) ) { |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 312 | VG_(printf_xml_no_f_c)(" <sframe> <fun>%t</fun> </sframe>\n", buf); |
| 313 | } else |
| 314 | if ( VG_(get_objname)(ip, buf, ERRTXT_LEN) ) { |
| 315 | VG_(printf_xml_no_f_c)(" <sframe> <obj>%t</obj> </sframe>\n", buf); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 316 | } else { |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 317 | VG_(printf_xml_no_f_c)(" <sframe> <obj>*</obj> </sframe>\n"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 318 | } |
| 319 | } |
| 320 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 321 | static void printSuppForIp_nonXML(UInt n, Addr ip, void* textV) |
| 322 | { |
| 323 | static UChar buf[ERRTXT_LEN]; |
| 324 | XArray* /* of HChar */ text = (XArray*)textV; |
| 325 | if ( VG_(get_fnname_no_cxx_demangle) (ip, buf, ERRTXT_LEN) ) { |
| 326 | VG_(xaprintf)(text, " fun:%s\n", buf); |
| 327 | } else |
| 328 | if ( VG_(get_objname)(ip, buf, ERRTXT_LEN) ) { |
| 329 | VG_(xaprintf)(text, " obj:%s\n", buf); |
| 330 | } else { |
| 331 | VG_(xaprintf)(text, " obj:*\n"); |
| 332 | } |
| 333 | } |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 334 | |
| 335 | /* Generate a suppression for an error, either in text or XML mode. |
| 336 | */ |
| 337 | static void gen_suppression(Error* err) |
| 338 | { |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 339 | Char xtra[256]; /* assumed big enough (is overrun-safe) */ |
| 340 | Bool anyXtra; |
| 341 | Char* name; |
| 342 | ExeContext* ec; |
| 343 | XArray* /* HChar */ text; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 344 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 345 | const HChar* dummy_name = "insert_a_suppression_name_here"; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 346 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 347 | vg_assert(err); |
| 348 | |
| 349 | /* In XML mode, we also need to print the plain text version of the |
| 350 | suppresion in a CDATA section. What that really means is, we |
| 351 | need to generate the plaintext version both in XML and text |
| 352 | mode. So generate it into TEXT. */ |
| 353 | text = VG_(newXA)( VG_(malloc), "errormgr.gen_suppression.1", |
| 354 | VG_(free), sizeof(HChar) ); |
| 355 | vg_assert(text); |
| 356 | |
| 357 | ec = VG_(get_error_where)(err); |
| 358 | vg_assert(ec); |
| 359 | |
| 360 | name = VG_TDICT_CALL(tool_get_error_name, err); |
| 361 | if (NULL == name) { |
| 362 | VG_(umsg)("(%s does not allow error to be suppressed)\n", |
| 363 | VG_(details).name); |
| 364 | return; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 365 | } |
| 366 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 367 | /* Ok. Generate the plain text version into TEXT. */ |
| 368 | VG_(xaprintf)(text, "{\n"); |
| 369 | VG_(xaprintf)(text, " <%s>\n", dummy_name); |
| 370 | VG_(xaprintf)(text, " %s:%s\n", VG_(details).name, name); |
| 371 | |
| 372 | VG_(memset)(xtra, 0, sizeof(xtra)); |
| 373 | anyXtra = VG_TDICT_CALL(tool_get_extra_suppression_info, |
| 374 | err, xtra, sizeof(xtra)); |
| 375 | vg_assert(xtra[sizeof(xtra)-1] == 0); |
| 376 | |
| 377 | if (anyXtra) |
| 378 | VG_(xaprintf)(text, " %s\n", xtra); |
| 379 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 380 | // Print stack trace elements |
sewardj | 2f340eb | 2011-01-28 00:44:52 +0000 | [diff] [blame] | 381 | UInt n_ips = VG_(get_ExeContext_n_ips)(ec); |
| 382 | tl_assert(n_ips > 0); |
| 383 | if (n_ips > VG_MAX_SUPP_CALLERS) |
| 384 | n_ips = VG_MAX_SUPP_CALLERS; |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 385 | VG_(apply_StackTrace)(printSuppForIp_nonXML, |
| 386 | text, |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 387 | VG_(get_ExeContext_StackTrace)(ec), |
sewardj | 2f340eb | 2011-01-28 00:44:52 +0000 | [diff] [blame] | 388 | n_ips); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 389 | |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 390 | VG_(xaprintf)(text, "}\n"); |
| 391 | // zero terminate |
| 392 | VG_(xaprintf)(text, "%c", (HChar)0 ); |
| 393 | // VG_(printf) of text |
| 394 | |
| 395 | /* And now display it. */ |
| 396 | if (! VG_(clo_xml) ) { |
| 397 | |
| 398 | // the simple case |
| 399 | VG_(printf)("%s", (HChar*) VG_(indexXA)(text, 0) ); |
| 400 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 401 | } else { |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 402 | |
| 403 | /* Now we have to print the XML directly. No need to go to the |
| 404 | effort of stuffing it in an XArray, since we won't need it |
| 405 | again. */ |
| 406 | VG_(printf_xml)(" <suppression>\n"); |
| 407 | VG_(printf_xml)(" <sname>%s</sname>\n", dummy_name); |
| 408 | VG_(printf_xml_no_f_c)( |
| 409 | " <skind>%t:%t</skind>\n", VG_(details).name, name); |
| 410 | if (anyXtra) |
| 411 | VG_(printf_xml_no_f_c)(" <skaux>%t</skaux>\n", xtra); |
| 412 | |
| 413 | // Print stack trace elements |
| 414 | VG_(apply_StackTrace)(printSuppForIp_XML, |
| 415 | NULL, |
| 416 | VG_(get_ExeContext_StackTrace)(ec), |
| 417 | VG_(get_ExeContext_n_ips)(ec)); |
| 418 | |
| 419 | // And now the cdata bit |
| 420 | // XXX FIXME! properly handle the case where the raw text |
| 421 | // itself contains "]]>", as specified in Protocol 4. |
| 422 | VG_(printf_xml)(" <rawtext>\n"); |
| 423 | VG_(printf_xml)("<![CDATA[\n"); |
sewardj | de78f9a | 2009-08-15 23:33:04 +0000 | [diff] [blame] | 424 | VG_(printf_xml)("%s", (HChar*) VG_(indexXA)(text, 0) ); |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 425 | VG_(printf_xml)("]]>\n"); |
| 426 | VG_(printf_xml)(" </rawtext>\n"); |
| 427 | VG_(printf_xml)(" </suppression>\n"); |
| 428 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 429 | } |
sewardj | 588adef | 2009-08-15 22:41:51 +0000 | [diff] [blame] | 430 | |
| 431 | VG_(deleteXA)(text); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 432 | } |
| 433 | |
| 434 | |
| 435 | /* Figure out if we want to perform a given action for this error, |
| 436 | possibly by asking the user. |
| 437 | */ |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 438 | Bool VG_(is_action_requested) ( Char* action, Bool* clo ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 439 | { |
| 440 | Char ch, ch2; |
| 441 | Int res; |
| 442 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 443 | /* First off, we shouldn't be asking the user anything if |
| 444 | we're in XML mode. */ |
| 445 | if (VG_(clo_xml)) |
| 446 | return False; /* That's a Nein, oder Nay as they say down here in B-W */ |
| 447 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 448 | if (*clo == False) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 449 | return False; |
| 450 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 451 | VG_(umsg)("\n"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 452 | |
| 453 | again: |
| 454 | VG_(printf)( |
| 455 | "==%d== " |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 456 | "---- %s ? --- [Return/N/n/Y/y/C/c] ---- ", |
| 457 | VG_(getpid)(), action |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 458 | ); |
| 459 | |
sewardj | 6024b21 | 2003-07-13 10:54:33 +0000 | [diff] [blame] | 460 | res = VG_(read)(VG_(clo_input_fd), &ch, 1); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 461 | if (res != 1) goto ioerror; |
| 462 | /* res == 1 */ |
| 463 | if (ch == '\n') return False; |
| 464 | if (ch != 'N' && ch != 'n' && ch != 'Y' && ch != 'y' |
| 465 | && ch != 'C' && ch != 'c') goto again; |
| 466 | |
sewardj | 6024b21 | 2003-07-13 10:54:33 +0000 | [diff] [blame] | 467 | res = VG_(read)(VG_(clo_input_fd), &ch2, 1); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 468 | if (res != 1) goto ioerror; |
| 469 | if (ch2 != '\n') goto again; |
| 470 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 471 | /* No, don't want to do action. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 472 | if (ch == 'n' || ch == 'N') return False; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 473 | /* Yes, want to do action. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 474 | if (ch == 'y' || ch == 'Y') return True; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 475 | /* No, don't want to do action, and don't ask again either. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 476 | vg_assert(ch == 'c' || ch == 'C'); |
| 477 | |
| 478 | ioerror: |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 479 | *clo = False; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 480 | return False; |
| 481 | } |
| 482 | |
| 483 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 484 | /* Do text-mode actions on error, that is, immediately after an error |
| 485 | is printed. These are: |
| 486 | * possibly, attach to a debugger |
| 487 | * possibly, generate a suppression. |
| 488 | Note this should not be called in XML mode! |
| 489 | */ |
| 490 | static |
| 491 | void do_actions_on_error(Error* err, Bool allow_db_attach) |
| 492 | { |
| 493 | Bool still_noisy = True; |
| 494 | |
| 495 | /* Should be assured by caller */ |
| 496 | vg_assert( ! VG_(clo_xml) ); |
| 497 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 498 | /* if user wants to debug from a certain error nr, then wait for gdb/vgdb */ |
| 499 | if (VG_(clo_vgdb) != Vg_VgdbNo |
| 500 | && allow_db_attach |
| 501 | && VG_(dyn_vgdb_error) <= n_errs_found) { |
| 502 | VG_(umsg)("(action on error) vgdb me ... \n"); |
| 503 | VG_(gdbserver)( err->tid ); |
| 504 | VG_(umsg)("Continuing ...\n"); |
| 505 | } |
| 506 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 507 | /* Perhaps we want a debugger attach at this point? */ |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 508 | /* GDBTD ??? maybe we should/could remove the below assuming the |
| 509 | gdbserver interface is better ??? */ |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 510 | if (allow_db_attach && |
| 511 | VG_(is_action_requested)( "Attach to debugger", & VG_(clo_db_attach) )) |
| 512 | { |
| 513 | if (0) VG_(printf)("starting debugger\n"); |
| 514 | VG_(start_debugger)( err->tid ); |
| 515 | } |
| 516 | /* Or maybe we want to generate the error's suppression? */ |
| 517 | if (VG_(clo_gen_suppressions) == 2 |
| 518 | || (VG_(clo_gen_suppressions) == 1 |
| 519 | && VG_(is_action_requested)( "Print suppression", &still_noisy )) |
| 520 | ) { |
| 521 | gen_suppression(err); |
| 522 | } |
| 523 | if (VG_(clo_gen_suppressions) == 1 && !still_noisy) |
| 524 | VG_(clo_gen_suppressions) = 0; |
| 525 | } |
| 526 | |
| 527 | |
| 528 | /* Prints an error. Not entirely simple because of the differences |
| 529 | between XML and text mode output. |
| 530 | |
| 531 | In XML mode: |
| 532 | |
| 533 | * calls the tool's pre-show method, so the tool can create any |
| 534 | preamble ahead of the message, if it wants. |
| 535 | |
| 536 | * prints the opening tag, and the <unique> and <tid> fields |
| 537 | |
| 538 | * prints the tool-specific parts of the message |
| 539 | |
| 540 | * if suppression generation is required, a suppression |
| 541 | |
| 542 | * the closing tag |
| 543 | |
| 544 | In text mode: |
| 545 | |
| 546 | * calls the tool's pre-show method, so the tool can create any |
| 547 | preamble ahead of the message, if it wants. |
| 548 | |
| 549 | * prints the tool-specific parts of the message |
| 550 | |
| 551 | * calls do_actions_on_error. This optionally does a debugger |
| 552 | attach (and detach), and optionally prints a suppression; both |
| 553 | of these may require user input. |
| 554 | */ |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 555 | static void pp_Error ( Error* err, Bool allow_db_attach, Bool xml ) |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 556 | { |
| 557 | /* If this fails, you probably specified your tool's method |
| 558 | dictionary incorrectly. */ |
| 559 | vg_assert(VG_(needs).tool_errors); |
| 560 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 561 | if (xml) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 562 | |
| 563 | /* Note, allow_db_attach is ignored in here. */ |
| 564 | |
| 565 | /* Ensure that suppression generation is either completely |
| 566 | enabled or completely disabled; either way, we won't require |
| 567 | any user input. m_main.process_cmd_line_options should |
| 568 | ensure the asserted condition holds. */ |
| 569 | vg_assert( VG_(clo_gen_suppressions) == 0 /* disabled */ |
| 570 | || VG_(clo_gen_suppressions) == 2 /* for all errors */ ); |
| 571 | |
| 572 | /* Pre-show it to the tool */ |
| 573 | VG_TDICT_CALL( tool_before_pp_Error, err ); |
| 574 | |
| 575 | /* standard preamble */ |
| 576 | VG_(printf_xml)("<error>\n"); |
| 577 | VG_(printf_xml)(" <unique>0x%x</unique>\n", err->unique); |
| 578 | VG_(printf_xml)(" <tid>%d</tid>\n", err->tid); |
| 579 | |
| 580 | /* actually print it */ |
| 581 | VG_TDICT_CALL( tool_pp_Error, err ); |
| 582 | |
| 583 | if (VG_(clo_gen_suppressions) > 0) |
| 584 | gen_suppression(err); |
| 585 | |
| 586 | /* postamble */ |
| 587 | VG_(printf_xml)("</error>\n"); |
njn | b6267bd | 2009-08-12 00:14:16 +0000 | [diff] [blame] | 588 | VG_(printf_xml)("\n"); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 589 | |
| 590 | } else { |
| 591 | |
| 592 | VG_TDICT_CALL( tool_before_pp_Error, err ); |
| 593 | |
| 594 | if (VG_(tdict).tool_show_ThreadIDs_for_errors |
| 595 | && err->tid > 0 && err->tid != last_tid_printed) { |
| 596 | VG_(umsg)("Thread %d:\n", err->tid ); |
| 597 | last_tid_printed = err->tid; |
| 598 | } |
| 599 | |
| 600 | VG_TDICT_CALL( tool_pp_Error, err ); |
njn | b6267bd | 2009-08-12 00:14:16 +0000 | [diff] [blame] | 601 | VG_(umsg)("\n"); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 602 | |
| 603 | do_actions_on_error(err, allow_db_attach); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 604 | } |
| 605 | } |
| 606 | |
| 607 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 608 | /* Construct an error */ |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 609 | static |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 610 | void construct_error ( Error* err, ThreadId tid, ErrorKind ekind, Addr a, |
| 611 | Char* s, void* extra, ExeContext* where ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 612 | { |
sewardj | dbada27 | 2005-07-02 21:16:30 +0000 | [diff] [blame] | 613 | /* DO NOT MAKE unique_counter NON-STATIC */ |
| 614 | static UInt unique_counter = 0; |
| 615 | |
njn | ca82cc0 | 2004-11-22 17:18:48 +0000 | [diff] [blame] | 616 | tl_assert(tid < VG_N_THREADS); |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 617 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 618 | /* Core-only parts */ |
sewardj | dbada27 | 2005-07-02 21:16:30 +0000 | [diff] [blame] | 619 | err->unique = unique_counter++; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 620 | err->next = NULL; |
| 621 | err->supp = NULL; |
| 622 | err->count = 1; |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 623 | err->tid = tid; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 624 | if (NULL == where) |
sewardj | adb102f | 2007-11-09 23:21:44 +0000 | [diff] [blame] | 625 | err->where = VG_(record_ExeContext)( tid, 0 ); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 626 | else |
| 627 | err->where = where; |
njn | 1d6c4bc | 2002-11-21 13:38:08 +0000 | [diff] [blame] | 628 | |
nethercote | 996901a | 2004-08-03 13:29:09 +0000 | [diff] [blame] | 629 | /* Tool-relevant parts */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 630 | err->ekind = ekind; |
| 631 | err->addr = a; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 632 | err->extra = extra; |
sewardj | a602261 | 2003-07-24 23:50:17 +0000 | [diff] [blame] | 633 | err->string = s; |
| 634 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 635 | /* sanity... */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 636 | vg_assert( tid < VG_N_THREADS ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 637 | } |
| 638 | |
njn | 83f9e79 | 2005-06-11 05:04:09 +0000 | [diff] [blame] | 639 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 640 | |
njn | fb289bc | 2007-01-12 23:59:50 +0000 | [diff] [blame] | 641 | static Int n_errs_shown = 0; |
| 642 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 643 | /* Top-level entry point to the error management subsystem. |
| 644 | All detected errors are notified here; this routine decides if/when the |
| 645 | user should see the error. */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 646 | void VG_(maybe_record_error) ( ThreadId tid, |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 647 | ErrorKind ekind, Addr a, Char* s, void* extra ) |
| 648 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 649 | Error err; |
| 650 | Error* p; |
| 651 | Error* p_prev; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 652 | UInt extra_size; |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 653 | VgRes exe_res = Vg_MedRes; |
| 654 | static Bool stopping_message = False; |
| 655 | static Bool slowdown_message = False; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 656 | |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 657 | /* After M_COLLECT_NO_ERRORS_AFTER_SHOWN different errors have |
| 658 | been found, or M_COLLECT_NO_ERRORS_AFTER_FOUND total errors |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 659 | have been found, just refuse to collect any more. This stops |
| 660 | the burden of the error-management system becoming excessive in |
| 661 | extremely buggy programs, although it does make it pretty |
| 662 | pointless to continue the Valgrind run after this point. */ |
sewardj | 2e43290 | 2002-06-13 20:44:00 +0000 | [diff] [blame] | 663 | if (VG_(clo_error_limit) |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 664 | && (n_errs_shown >= M_COLLECT_NO_ERRORS_AFTER_SHOWN |
sewardj | 8a05172 | 2005-06-30 00:10:16 +0000 | [diff] [blame] | 665 | || n_errs_found >= M_COLLECT_NO_ERRORS_AFTER_FOUND) |
| 666 | && !VG_(clo_xml)) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 667 | if (!stopping_message) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 668 | VG_(umsg)("\n"); |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 669 | |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 670 | if (n_errs_shown >= M_COLLECT_NO_ERRORS_AFTER_SHOWN) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 671 | VG_(umsg)( |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 672 | "More than %d different errors detected. " |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 673 | "I'm not reporting any more.\n", |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 674 | M_COLLECT_NO_ERRORS_AFTER_SHOWN ); |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 675 | } else { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 676 | VG_(umsg)( |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 677 | "More than %d total errors detected. " |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 678 | "I'm not reporting any more.\n", |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 679 | M_COLLECT_NO_ERRORS_AFTER_FOUND ); |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 680 | } |
| 681 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 682 | VG_(umsg)("Final error counts will be inaccurate. " |
| 683 | "Go fix your program!\n"); |
| 684 | VG_(umsg)("Rerun with --error-limit=no to disable " |
| 685 | "this cutoff. Note\n"); |
| 686 | VG_(umsg)("that errors may occur in your program without " |
| 687 | "prior warning from\n"); |
| 688 | VG_(umsg)("Valgrind, because errors are no longer " |
| 689 | "being displayed.\n"); |
| 690 | VG_(umsg)("\n"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 691 | stopping_message = True; |
| 692 | } |
| 693 | return; |
| 694 | } |
| 695 | |
njn | 14319cc | 2005-03-13 06:26:22 +0000 | [diff] [blame] | 696 | /* After M_COLLECT_ERRORS_SLOWLY_AFTER different errors have |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 697 | been found, be much more conservative about collecting new |
| 698 | ones. */ |
sewardj | 8a05172 | 2005-06-30 00:10:16 +0000 | [diff] [blame] | 699 | if (n_errs_shown >= M_COLLECT_ERRORS_SLOWLY_AFTER |
| 700 | && !VG_(clo_xml)) { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 701 | exe_res = Vg_LowRes; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 702 | if (!slowdown_message) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 703 | VG_(umsg)("\n"); |
| 704 | VG_(umsg)("More than %d errors detected. Subsequent errors\n", |
| 705 | M_COLLECT_ERRORS_SLOWLY_AFTER); |
| 706 | VG_(umsg)("will still be recorded, but in less " |
| 707 | "detail than before.\n"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 708 | slowdown_message = True; |
| 709 | } |
| 710 | } |
| 711 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 712 | /* Build ourselves the error */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 713 | construct_error ( &err, tid, ekind, a, s, extra, NULL ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 714 | |
| 715 | /* First, see if we've got an error record matching this one. */ |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 716 | em_errlist_searches++; |
| 717 | p = errors; |
| 718 | p_prev = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 719 | while (p != NULL) { |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 720 | em_errlist_cmps++; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 721 | if (eq_Error(exe_res, p, &err)) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 722 | /* Found it. */ |
| 723 | p->count++; |
| 724 | if (p->supp != NULL) { |
| 725 | /* Deal correctly with suppressed errors. */ |
| 726 | p->supp->count++; |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 727 | n_errs_suppressed++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 728 | } else { |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 729 | n_errs_found++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 730 | } |
| 731 | |
| 732 | /* Move p to the front of the list so that future searches |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 733 | for it are faster. It also allows to print the last |
| 734 | error (see VG_(show_last_error). */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 735 | if (p_prev != NULL) { |
| 736 | vg_assert(p_prev->next == p); |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 737 | p_prev->next = p->next; |
| 738 | p->next = errors; |
| 739 | errors = p; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 740 | } |
sewardj | 7ebf7c3 | 2003-07-24 21:29:40 +0000 | [diff] [blame] | 741 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 742 | return; |
| 743 | } |
| 744 | p_prev = p; |
| 745 | p = p->next; |
| 746 | } |
| 747 | |
| 748 | /* Didn't see it. Copy and add. */ |
| 749 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 750 | /* OK, we're really going to collect it. The context is on the stack and |
| 751 | will disappear shortly, so we must copy it. First do the main |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 752 | (non-'extra') part. |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 753 | |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 754 | Then VG_(tdict).tool_update_extra can update the 'extra' part. This |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 755 | is for when there are more details to fill in which take time to work |
| 756 | out but don't affect our earlier decision to include the error -- by |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 757 | postponing those details until now, we avoid the extra work in the |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 758 | case where we ignore the error. Ugly. |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 759 | |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 760 | Then, if there is an 'extra' part, copy it too, using the size that |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 761 | VG_(tdict).tool_update_extra returned. Also allow for people using |
| 762 | the void* extra field for a scalar value like an integer. |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 763 | */ |
| 764 | |
| 765 | /* copy main part */ |
sewardj | 9c606bd | 2008-09-18 18:12:50 +0000 | [diff] [blame] | 766 | p = VG_(arena_malloc)(VG_AR_ERRORS, "errormgr.mre.1", sizeof(Error)); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 767 | *p = err; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 768 | |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 769 | /* update 'extra' */ |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 770 | switch (ekind) { |
njn | a86f29e | 2006-12-14 02:55:58 +0000 | [diff] [blame] | 771 | //(example code, see comment on CoreSuppKind above) |
| 772 | //case ThreadErr: |
| 773 | // vg_assert(VG_(needs).core_errors); |
| 774 | // extra_size = <something> |
| 775 | // break; |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 776 | default: |
| 777 | vg_assert(VG_(needs).tool_errors); |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 778 | extra_size = VG_TDICT_CALL(tool_update_extra, p); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 779 | break; |
| 780 | } |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 781 | |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 782 | /* copy block pointed to by 'extra', if there is one */ |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 783 | if (NULL != p->extra && 0 != extra_size) { |
sewardj | 9c606bd | 2008-09-18 18:12:50 +0000 | [diff] [blame] | 784 | void* new_extra = VG_(malloc)("errormgr.mre.2", extra_size); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 785 | VG_(memcpy)(new_extra, p->extra, extra_size); |
| 786 | p->extra = new_extra; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 787 | } |
| 788 | |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 789 | p->next = errors; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 790 | p->supp = is_suppressible_error(&err); |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 791 | errors = p; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 792 | if (p->supp == NULL) { |
njn | 606a4ae | 2009-08-11 00:52:40 +0000 | [diff] [blame] | 793 | n_err_contexts++; |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 794 | n_errs_found++; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 795 | /* Actually show the error; more complex than you might think. */ |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 796 | pp_Error( p, /*allow_db_attach*/True, VG_(clo_xml) ); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 797 | /* update stats */ |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 798 | n_errs_shown++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 799 | } else { |
njn | 606a4ae | 2009-08-11 00:52:40 +0000 | [diff] [blame] | 800 | n_supp_contexts++; |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 801 | n_errs_suppressed++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 802 | p->supp->count++; |
| 803 | } |
| 804 | } |
| 805 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 806 | /* Second top-level entry point to the error management subsystem, for |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 807 | errors that the tool wants to report immediately, eg. because they're |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 808 | guaranteed to only happen once. This avoids all the recording and |
| 809 | comparing stuff. But they can be suppressed; returns True if it is |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 810 | suppressed. Bool 'print_error' dictates whether to print the error. |
njn | 18afe5d | 2009-08-10 08:25:39 +0000 | [diff] [blame] | 811 | Bool 'count_error' dictates whether to count the error in n_errs_found. |
njn | 47363ab | 2003-04-21 13:24:40 +0000 | [diff] [blame] | 812 | */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 813 | Bool VG_(unique_error) ( ThreadId tid, ErrorKind ekind, Addr a, Char* s, |
njn | 3e88418 | 2003-04-15 13:03:23 +0000 | [diff] [blame] | 814 | void* extra, ExeContext* where, Bool print_error, |
njn | 18afe5d | 2009-08-10 08:25:39 +0000 | [diff] [blame] | 815 | Bool allow_db_attach, Bool count_error ) |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 816 | { |
njn | fb289bc | 2007-01-12 23:59:50 +0000 | [diff] [blame] | 817 | Error err; |
| 818 | Supp *su; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 819 | |
| 820 | /* Build ourselves the error */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 821 | construct_error ( &err, tid, ekind, a, s, extra, where ); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 822 | |
| 823 | /* Unless it's suppressed, we're going to show it. Don't need to make |
| 824 | a copy, because it's only temporary anyway. |
| 825 | |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 826 | Then update the 'extra' part with VG_(tdict).tool_update_extra), |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 827 | because that can have an affect on whether it's suppressed. Ignore |
| 828 | the size return value of VG_(tdict).tool_update_extra, because we're |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 829 | not copying 'extra'. */ |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 830 | (void)VG_TDICT_CALL(tool_update_extra, &err); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 831 | |
njn | fb289bc | 2007-01-12 23:59:50 +0000 | [diff] [blame] | 832 | su = is_suppressible_error(&err); |
| 833 | if (NULL == su) { |
njn | 606a4ae | 2009-08-11 00:52:40 +0000 | [diff] [blame] | 834 | if (count_error) { |
njn | 18afe5d | 2009-08-10 08:25:39 +0000 | [diff] [blame] | 835 | n_errs_found++; |
njn | 606a4ae | 2009-08-11 00:52:40 +0000 | [diff] [blame] | 836 | n_err_contexts++; |
| 837 | } |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 838 | |
| 839 | if (print_error) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 840 | /* Actually show the error; more complex than you might think. */ |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 841 | pp_Error(&err, allow_db_attach, VG_(clo_xml)); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 842 | /* update stats */ |
njn | fb289bc | 2007-01-12 23:59:50 +0000 | [diff] [blame] | 843 | n_errs_shown++; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 844 | } |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 845 | return False; |
| 846 | |
| 847 | } else { |
njn | 663ab79 | 2009-08-13 04:24:38 +0000 | [diff] [blame] | 848 | if (count_error) { |
| 849 | n_errs_suppressed++; |
| 850 | n_supp_contexts++; |
| 851 | } |
njn | fb289bc | 2007-01-12 23:59:50 +0000 | [diff] [blame] | 852 | su->count++; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 853 | return True; |
| 854 | } |
| 855 | } |
| 856 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 857 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 858 | /*------------------------------------------------------------*/ |
| 859 | /*--- Exported fns ---*/ |
| 860 | /*------------------------------------------------------------*/ |
| 861 | |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 862 | /* Show the used suppressions. Returns False if no suppression |
| 863 | got used. */ |
| 864 | static Bool show_used_suppressions ( void ) |
| 865 | { |
| 866 | Supp *su; |
| 867 | Bool any_supp; |
| 868 | |
sewardj | 7c9e57c | 2005-05-24 14:21:45 +0000 | [diff] [blame] | 869 | if (VG_(clo_xml)) |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 870 | VG_(printf_xml)("<suppcounts>\n"); |
sewardj | 7c9e57c | 2005-05-24 14:21:45 +0000 | [diff] [blame] | 871 | |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 872 | any_supp = False; |
| 873 | for (su = suppressions; su != NULL; su = su->next) { |
| 874 | if (su->count <= 0) |
| 875 | continue; |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 876 | if (VG_(clo_xml)) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 877 | VG_(printf_xml_no_f_c)( " <pair>\n" |
| 878 | " <count>%d</count>\n" |
| 879 | " <name>%t</name>\n" |
| 880 | " </pair>\n", |
| 881 | su->count, su->sname ); |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 882 | } else { |
sewardj | 2d9e874 | 2009-08-07 15:46:56 +0000 | [diff] [blame] | 883 | // blank line before the first shown suppression, if any |
| 884 | if (!any_supp) |
| 885 | VG_(dmsg)("\n"); |
| 886 | VG_(dmsg)("used_suppression: %6d %s\n", su->count, su->sname); |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 887 | } |
sewardj | 2d9e874 | 2009-08-07 15:46:56 +0000 | [diff] [blame] | 888 | any_supp = True; |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 889 | } |
| 890 | |
sewardj | 7c9e57c | 2005-05-24 14:21:45 +0000 | [diff] [blame] | 891 | if (VG_(clo_xml)) |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 892 | VG_(printf_xml)("</suppcounts>\n"); |
sewardj | 7c9e57c | 2005-05-24 14:21:45 +0000 | [diff] [blame] | 893 | |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 894 | return any_supp; |
| 895 | } |
| 896 | |
sewardj | 9f297ca | 2005-05-20 02:29:52 +0000 | [diff] [blame] | 897 | /* Show all the errors that occurred, and possibly also the |
| 898 | suppressions used. */ |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 899 | void VG_(show_all_errors) ( Int verbosity, Bool xml ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 900 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 901 | Int i, n_min; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 902 | Error *p, *p_min; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 903 | Bool any_supp; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 904 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 905 | if (verbosity == 0) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 906 | return; |
| 907 | |
njn | 606a4ae | 2009-08-11 00:52:40 +0000 | [diff] [blame] | 908 | /* If we're printing XML, just show the suppressions and stop. */ |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 909 | if (xml) { |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 910 | (void)show_used_suppressions(); |
| 911 | return; |
| 912 | } |
| 913 | |
| 914 | /* We only get here if not printing XML. */ |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 915 | VG_(umsg)("ERROR SUMMARY: " |
| 916 | "%d errors from %d contexts (suppressed: %d from %d)\n", |
| 917 | n_errs_found, n_err_contexts, |
| 918 | n_errs_suppressed, n_supp_contexts ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 919 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 920 | if (verbosity <= 1) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 921 | return; |
| 922 | |
sewardj | 2d9e874 | 2009-08-07 15:46:56 +0000 | [diff] [blame] | 923 | // We do the following only at -v or above, and only in non-XML |
| 924 | // mode |
| 925 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 926 | /* Print the contexts in order of increasing error count. |
| 927 | Once an error is shown, we add a huge value to its count to filter it |
| 928 | out. After having shown all errors, we reset count to the original value. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 929 | for (i = 0; i < n_err_contexts; i++) { |
| 930 | n_min = (1 << 30) - 1; |
| 931 | p_min = NULL; |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 932 | for (p = errors; p != NULL; p = p->next) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 933 | if (p->supp != NULL) continue; |
| 934 | if (p->count < n_min) { |
| 935 | n_min = p->count; |
| 936 | p_min = p; |
| 937 | } |
| 938 | } |
njn | 663ab79 | 2009-08-13 04:24:38 +0000 | [diff] [blame] | 939 | // XXX: this isn't right. See bug 203651. |
| 940 | if (p_min == NULL) continue; //VG_(tool_panic)("show_all_errors()"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 941 | |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 942 | VG_(umsg)("\n"); |
| 943 | VG_(umsg)("%d errors in context %d of %d:\n", |
| 944 | p_min->count, i+1, n_err_contexts); |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 945 | pp_Error( p_min, False/*allow_db_attach*/, False /* xml */ ); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 946 | |
| 947 | // We're not printing XML -- we'd have exited above if so. |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 948 | vg_assert(! xml); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 949 | |
| 950 | if ((i+1 == VG_(clo_dump_error))) { |
sewardj | 7cf4e6b | 2008-05-01 20:24:26 +0000 | [diff] [blame] | 951 | StackTrace ips = VG_(get_ExeContext_StackTrace)(p_min->where); |
sewardj | fa8ec11 | 2005-01-19 11:55:34 +0000 | [diff] [blame] | 952 | VG_(translate) ( 0 /* dummy ThreadId; irrelevant due to debugging*/, |
njn | 394213a | 2005-06-19 18:38:24 +0000 | [diff] [blame] | 953 | ips[0], /*debugging*/True, 0xFE/*verbosity*/, |
sewardj | 0ec07f3 | 2006-01-12 12:32:32 +0000 | [diff] [blame] | 954 | /*bbs_done*/0, |
| 955 | /*allow redir?*/True); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 956 | } |
| 957 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 958 | p_min->count = p_min->count + (1 << 30); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 959 | } |
| 960 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 961 | /* reset the counts, otherwise a 2nd call does not show anything anymore */ |
| 962 | for (p = errors; p != NULL; p = p->next) { |
| 963 | if (p->count >= (1 << 30)) |
| 964 | p->count = p->count - (1 << 30); |
| 965 | } |
| 966 | |
| 967 | |
sewardj | 71bc3cb | 2005-05-19 00:25:45 +0000 | [diff] [blame] | 968 | any_supp = show_used_suppressions(); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 969 | |
sewardj | 2d9e874 | 2009-08-07 15:46:56 +0000 | [diff] [blame] | 970 | if (any_supp) |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 971 | VG_(umsg)("\n"); |
sewardj | 2d9e874 | 2009-08-07 15:46:56 +0000 | [diff] [blame] | 972 | // reprint this, so users don't have to scroll way up to find |
| 973 | // the first printing |
| 974 | VG_(umsg)("ERROR SUMMARY: " |
| 975 | "%d errors from %d contexts (suppressed: %d from %d)\n", |
| 976 | n_errs_found, n_err_contexts, n_errs_suppressed, |
| 977 | n_supp_contexts ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 978 | } |
| 979 | |
sewardj | 3b29048 | 2011-05-06 21:02:55 +0000 | [diff] [blame] | 980 | void VG_(show_last_error) ( void ) |
| 981 | { |
| 982 | if (n_err_contexts == 0) { |
| 983 | VG_(umsg)("No errors yet\n"); |
| 984 | return; |
| 985 | } |
| 986 | |
| 987 | pp_Error( errors, False/*allow_db_attach*/, False/*xml*/ ); |
| 988 | } |
| 989 | |
sewardj | 9f297ca | 2005-05-20 02:29:52 +0000 | [diff] [blame] | 990 | |
| 991 | /* Show occurrence counts of all errors, in XML form. */ |
| 992 | void VG_(show_error_counts_as_XML) ( void ) |
| 993 | { |
| 994 | Error* err; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 995 | VG_(printf_xml)("<errorcounts>\n"); |
sewardj | 9f297ca | 2005-05-20 02:29:52 +0000 | [diff] [blame] | 996 | for (err = errors; err != NULL; err = err->next) { |
| 997 | if (err->supp != NULL) |
| 998 | continue; |
| 999 | if (err->count <= 0) |
| 1000 | continue; |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1001 | VG_(printf_xml)(" <pair>\n"); |
| 1002 | VG_(printf_xml)(" <count>%d</count>\n", err->count); |
| 1003 | VG_(printf_xml)(" <unique>0x%x</unique>\n", err->unique); |
| 1004 | VG_(printf_xml)(" </pair>\n"); |
sewardj | 9f297ca | 2005-05-20 02:29:52 +0000 | [diff] [blame] | 1005 | } |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1006 | VG_(printf_xml)("</errorcounts>\n"); |
njn | b6267bd | 2009-08-12 00:14:16 +0000 | [diff] [blame] | 1007 | VG_(printf_xml)("\n"); |
sewardj | 9f297ca | 2005-05-20 02:29:52 +0000 | [diff] [blame] | 1008 | } |
| 1009 | |
| 1010 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1011 | /*------------------------------------------------------------*/ |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1012 | /*--- Suppression parsing ---*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1013 | /*------------------------------------------------------------*/ |
| 1014 | |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1015 | /* Get the next char from fd into *out_buf. Returns 1 if success, |
| 1016 | 0 if eof or < 0 if error. */ |
| 1017 | |
| 1018 | static Int get_char ( Int fd, Char* out_buf ) |
| 1019 | { |
| 1020 | Int r; |
sewardj | 89c5bd8 | 2008-10-23 10:15:37 +0000 | [diff] [blame] | 1021 | static Char buf[256]; |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1022 | static Int buf_size = 0; |
| 1023 | static Int buf_used = 0; |
sewardj | 89c5bd8 | 2008-10-23 10:15:37 +0000 | [diff] [blame] | 1024 | vg_assert(buf_size >= 0 && buf_size <= 256); |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1025 | vg_assert(buf_used >= 0 && buf_used <= buf_size); |
| 1026 | if (buf_used == buf_size) { |
sewardj | 89c5bd8 | 2008-10-23 10:15:37 +0000 | [diff] [blame] | 1027 | r = VG_(read)(fd, buf, 256); |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1028 | if (r < 0) return r; /* read failed */ |
sewardj | 89c5bd8 | 2008-10-23 10:15:37 +0000 | [diff] [blame] | 1029 | vg_assert(r >= 0 && r <= 256); |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1030 | buf_size = r; |
| 1031 | buf_used = 0; |
| 1032 | } |
| 1033 | if (buf_size == 0) |
| 1034 | return 0; /* eof */ |
sewardj | 89c5bd8 | 2008-10-23 10:15:37 +0000 | [diff] [blame] | 1035 | vg_assert(buf_size >= 0 && buf_size <= 256); |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1036 | vg_assert(buf_used >= 0 && buf_used < buf_size); |
| 1037 | *out_buf = buf[buf_used]; |
| 1038 | buf_used++; |
| 1039 | return 1; |
| 1040 | } |
| 1041 | |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1042 | Bool VG_(get_line) ( Int fd, Char** bufpp, SizeT* nBufp, Int* lineno ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1043 | { |
njn | 35db56c | 2009-07-24 07:38:29 +0000 | [diff] [blame] | 1044 | Char* buf = *bufpp; |
| 1045 | SizeT nBuf = *nBufp; |
| 1046 | Char ch; |
| 1047 | Int n, i; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1048 | while (True) { |
| 1049 | /* First, read until a non-blank char appears. */ |
| 1050 | while (True) { |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1051 | n = get_char(fd, &ch); |
njn | 0c0f32a | 2005-03-26 04:14:01 +0000 | [diff] [blame] | 1052 | if (n == 1 && !VG_(isspace)(ch)) break; |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1053 | if (n == 1 && ch == '\n' && lineno) |
| 1054 | (*lineno)++; |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1055 | if (n <= 0) return True; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1056 | } |
| 1057 | |
| 1058 | /* Now, read the line into buf. */ |
| 1059 | i = 0; |
| 1060 | buf[i++] = ch; buf[i] = 0; |
| 1061 | while (True) { |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1062 | n = get_char(fd, &ch); |
| 1063 | if (n <= 0) return False; /* the next call will return True */ |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1064 | if (ch == '\n' && lineno) |
| 1065 | (*lineno)++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1066 | if (ch == '\n') break; |
njn | 35db56c | 2009-07-24 07:38:29 +0000 | [diff] [blame] | 1067 | if (i > 0 && i == nBuf-1) { |
| 1068 | *nBufp = nBuf = nBuf * 2; |
| 1069 | #define RIDICULOUS 100000 |
| 1070 | vg_assert2(nBuf < RIDICULOUS, // Just a sanity check, really. |
| 1071 | "VG_(get_line): line longer than %d chars, aborting\n", |
| 1072 | RIDICULOUS); |
| 1073 | *bufpp = buf = VG_(realloc)("errormgr.get_line.1", buf, nBuf); |
| 1074 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1075 | buf[i++] = ch; buf[i] = 0; |
| 1076 | } |
njn | 0c0f32a | 2005-03-26 04:14:01 +0000 | [diff] [blame] | 1077 | while (i > 1 && VG_(isspace)(buf[i-1])) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1078 | i--; buf[i] = 0; |
| 1079 | }; |
| 1080 | |
njn | 02bc4b8 | 2005-05-15 17:28:26 +0000 | [diff] [blame] | 1081 | /* VG_(printf)("The line is '%s'\n", buf); */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1082 | /* Ok, we have a line. If a non-comment line, return. |
| 1083 | If a comment line, start all over again. */ |
| 1084 | if (buf[0] != '#') return False; |
| 1085 | } |
| 1086 | } |
| 1087 | |
| 1088 | |
| 1089 | /* *p_caller contains the raw name of a caller, supposedly either |
| 1090 | fun:some_function_name or |
| 1091 | obj:some_object_name. |
| 1092 | Set *p_ty accordingly and advance *p_caller over the descriptor |
| 1093 | (fun: or obj:) part. |
| 1094 | Returns False if failed. |
| 1095 | */ |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1096 | static Bool setLocationTy ( SuppLoc* p ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1097 | { |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1098 | if (VG_(strncmp)(p->name, "fun:", 4) == 0) { |
| 1099 | p->name += 4; |
| 1100 | p->ty = FunName; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1101 | return True; |
| 1102 | } |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1103 | if (VG_(strncmp)(p->name, "obj:", 4) == 0) { |
| 1104 | p->name += 4; |
| 1105 | p->ty = ObjName; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1106 | return True; |
| 1107 | } |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 1108 | if (VG_(strcmp)(p->name, "...") == 0) { |
| 1109 | p->name = NULL; |
| 1110 | p->ty = DotDotDot; |
| 1111 | return True; |
| 1112 | } |
| 1113 | VG_(printf)("location should be \"...\", or should start " |
| 1114 | "with \"fun:\" or \"obj:\"\n"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1115 | return False; |
| 1116 | } |
| 1117 | |
| 1118 | |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 1119 | /* Look for "tool" in a string like "tool1,tool2,tool3" */ |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1120 | static Bool tool_name_present(Char *name, Char *names) |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 1121 | { |
| 1122 | Bool found; |
| 1123 | Char *s = NULL; /* Shut gcc up */ |
| 1124 | Int len = VG_(strlen)(name); |
| 1125 | |
| 1126 | found = (NULL != (s = VG_(strstr)(names, name)) && |
| 1127 | (s == names || *(s-1) == ',') && |
| 1128 | (*(s+len) == ',' || *(s+len) == '\0') |
| 1129 | ); |
| 1130 | |
| 1131 | return found; |
| 1132 | } |
| 1133 | |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 1134 | /* Read suppressions from the file specified in VG_(clo_suppressions) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1135 | and place them in the suppressions list. If there's any difficulty |
| 1136 | doing this, just give up -- there's no point in trying to recover. |
| 1137 | */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1138 | static void load_one_suppressions_file ( Char* filename ) |
| 1139 | { |
sewardj | 9264559 | 2005-07-23 09:18:34 +0000 | [diff] [blame] | 1140 | SysRes sres; |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 1141 | Int fd, i, j, lineno = 0; |
sewardj | 9264559 | 2005-07-23 09:18:34 +0000 | [diff] [blame] | 1142 | Bool eof; |
njn | 35db56c | 2009-07-24 07:38:29 +0000 | [diff] [blame] | 1143 | SizeT nBuf = 200; |
| 1144 | Char* buf = VG_(malloc)("errormgr.losf.1", nBuf); |
sewardj | 9264559 | 2005-07-23 09:18:34 +0000 | [diff] [blame] | 1145 | Char* tool_names; |
| 1146 | Char* supp_name; |
| 1147 | Char* err_str = NULL; |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1148 | SuppLoc tmp_callers[VG_MAX_SUPP_CALLERS]; |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1149 | |
njn | cc37d2e | 2009-06-24 03:49:19 +0000 | [diff] [blame] | 1150 | // Check it's not a directory. |
| 1151 | if (VG_(is_dir)( filename )) { |
| 1152 | if (VG_(clo_xml)) |
sewardj | 09361bf | 2011-01-28 00:19:25 +0000 | [diff] [blame] | 1153 | VG_(printf_xml)("</valgrindoutput>\n"); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1154 | VG_(umsg)("FATAL: suppressions file \"%s\" is a directory\n", filename ); |
njn | cc37d2e | 2009-06-24 03:49:19 +0000 | [diff] [blame] | 1155 | VG_(exit)(1); |
| 1156 | } |
| 1157 | |
| 1158 | // Open the suppression file. |
sewardj | 9264559 | 2005-07-23 09:18:34 +0000 | [diff] [blame] | 1159 | sres = VG_(open)( filename, VKI_O_RDONLY, 0 ); |
njn | cda2f0f | 2009-05-18 02:12:08 +0000 | [diff] [blame] | 1160 | if (sr_isError(sres)) { |
sewardj | f349d55 | 2005-11-14 17:01:01 +0000 | [diff] [blame] | 1161 | if (VG_(clo_xml)) |
sewardj | 09361bf | 2011-01-28 00:19:25 +0000 | [diff] [blame] | 1162 | VG_(printf_xml)("</valgrindoutput>\n"); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1163 | VG_(umsg)("FATAL: can't open suppressions file \"%s\"\n", filename ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1164 | VG_(exit)(1); |
| 1165 | } |
njn | cda2f0f | 2009-05-18 02:12:08 +0000 | [diff] [blame] | 1166 | fd = sr_Res(sres); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1167 | |
sewardj | 9264559 | 2005-07-23 09:18:34 +0000 | [diff] [blame] | 1168 | # define BOMB(S) { err_str = S; goto syntax_error; } |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1169 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1170 | while (True) { |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 1171 | /* Assign and initialise the two suppression halves (core and tool) */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1172 | Supp* supp; |
sewardj | 9c606bd | 2008-09-18 18:12:50 +0000 | [diff] [blame] | 1173 | supp = VG_(arena_malloc)(VG_AR_CORE, "errormgr.losf.1", |
| 1174 | sizeof(Supp)); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1175 | supp->count = 0; |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1176 | |
| 1177 | // Initialise temporary reading-in buffer. |
| 1178 | for (i = 0; i < VG_MAX_SUPP_CALLERS; i++) { |
| 1179 | tmp_callers[i].ty = NoName; |
| 1180 | tmp_callers[i].name = NULL; |
| 1181 | } |
| 1182 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1183 | supp->string = supp->extra = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1184 | |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1185 | eof = VG_(get_line) ( fd, &buf, &nBuf, &lineno ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1186 | if (eof) break; |
| 1187 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1188 | if (!VG_STREQ(buf, "{")) BOMB("expected '{' or end-of-file"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1189 | |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1190 | eof = VG_(get_line) ( fd, &buf, &nBuf, &lineno ); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1191 | |
| 1192 | if (eof || VG_STREQ(buf, "}")) BOMB("unexpected '}'"); |
| 1193 | |
sewardj | 9c606bd | 2008-09-18 18:12:50 +0000 | [diff] [blame] | 1194 | supp->sname = VG_(arena_strdup)(VG_AR_CORE, "errormgr.losf.2", buf); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1195 | |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1196 | eof = VG_(get_line) ( fd, &buf, &nBuf, &lineno ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1197 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1198 | if (eof) BOMB("unexpected end-of-file"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1199 | |
njn | 94065fd | 2004-11-22 19:26:27 +0000 | [diff] [blame] | 1200 | /* Check it has the "tool1,tool2,...:supp" form (look for ':') */ |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1201 | i = 0; |
| 1202 | while (True) { |
| 1203 | if (buf[i] == ':') break; |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1204 | if (buf[i] == '\0') BOMB("malformed 'tool1,tool2,...:supp' line"); |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1205 | i++; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1206 | } |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1207 | buf[i] = '\0'; /* Replace ':', splitting into two strings */ |
| 1208 | |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 1209 | tool_names = & buf[0]; |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 1210 | supp_name = & buf[i+1]; |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1211 | |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 1212 | if (VG_(needs).core_errors && tool_name_present("core", tool_names)) |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1213 | { |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1214 | // A core suppression |
njn | a86f29e | 2006-12-14 02:55:58 +0000 | [diff] [blame] | 1215 | //(example code, see comment on CoreSuppKind above) |
| 1216 | //if (VG_STREQ(supp_name, "Thread")) |
| 1217 | // supp->skind = ThreadSupp; |
| 1218 | //else |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1219 | BOMB("unknown core suppression type"); |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1220 | } |
njn | 95ec870 | 2004-11-22 16:46:13 +0000 | [diff] [blame] | 1221 | else if (VG_(needs).tool_errors && |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 1222 | tool_name_present(VG_(details).name, tool_names)) |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1223 | { |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1224 | // A tool suppression |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 1225 | if (VG_TDICT_CALL(tool_recognised_suppression, supp_name, supp)) { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1226 | /* Do nothing, function fills in supp->skind */ |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1227 | } else { |
| 1228 | BOMB("unknown tool suppression type"); |
| 1229 | } |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 1230 | } |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1231 | else { |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1232 | // Ignore rest of suppression |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1233 | while (True) { |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1234 | eof = VG_(get_line) ( fd, &buf, &nBuf, &lineno ); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1235 | if (eof) BOMB("unexpected end-of-file"); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 1236 | if (VG_STREQ(buf, "}")) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1237 | break; |
| 1238 | } |
| 1239 | continue; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1240 | } |
| 1241 | |
njn | 95ec870 | 2004-11-22 16:46:13 +0000 | [diff] [blame] | 1242 | if (VG_(needs).tool_errors && |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1243 | !VG_TDICT_CALL(tool_read_extra_suppression_info, |
njn | 35db56c | 2009-07-24 07:38:29 +0000 | [diff] [blame] | 1244 | fd, &buf, &nBuf, supp)) |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1245 | { |
| 1246 | BOMB("bad or missing extra suppression info"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1247 | } |
| 1248 | |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 1249 | /* the main frame-descriptor reading loop */ |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1250 | i = 0; |
| 1251 | while (True) { |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1252 | eof = VG_(get_line) ( fd, &buf, &nBuf, &lineno ); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1253 | if (eof) |
| 1254 | BOMB("unexpected end-of-file"); |
| 1255 | if (VG_STREQ(buf, "}")) { |
| 1256 | if (i > 0) { |
| 1257 | break; |
| 1258 | } else { |
| 1259 | BOMB("missing stack trace"); |
| 1260 | } |
| 1261 | } |
| 1262 | if (i == VG_MAX_SUPP_CALLERS) |
| 1263 | BOMB("too many callers in stack trace"); |
| 1264 | if (i > 0 && i >= VG_(clo_backtrace_size)) |
| 1265 | break; |
sewardj | 9c606bd | 2008-09-18 18:12:50 +0000 | [diff] [blame] | 1266 | tmp_callers[i].name = VG_(arena_strdup)(VG_AR_CORE, |
| 1267 | "errormgr.losf.3", buf); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1268 | if (!setLocationTy(&(tmp_callers[i]))) |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 1269 | BOMB("location should be \"...\", or should start " |
| 1270 | "with \"fun:\" or \"obj:\""); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1271 | i++; |
| 1272 | } |
| 1273 | |
| 1274 | // If the num callers is >= VG_(clo_backtrace_size), ignore any extra |
| 1275 | // lines and grab the '}'. |
sewardj | 57a8f5f | 2003-07-06 01:40:11 +0000 | [diff] [blame] | 1276 | if (!VG_STREQ(buf, "}")) { |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1277 | do { |
bart | 050eec5 | 2009-07-27 12:03:03 +0000 | [diff] [blame] | 1278 | eof = VG_(get_line) ( fd, &buf, &nBuf, &lineno ); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1279 | } while (!eof && !VG_STREQ(buf, "}")); |
| 1280 | } |
| 1281 | |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 1282 | // Reject entries which are entirely composed of frame |
| 1283 | // level wildcards. |
| 1284 | vg_assert(i > 0); // guaranteed by frame-descriptor reading loop |
| 1285 | for (j = 0; j < i; j++) { |
| 1286 | if (tmp_callers[j].ty == FunName || tmp_callers[j].ty == ObjName) |
| 1287 | break; |
| 1288 | vg_assert(tmp_callers[j].ty == DotDotDot); |
| 1289 | } |
| 1290 | vg_assert(j >= 0 && j <= i); |
| 1291 | if (j == i) { |
| 1292 | // we didn't find any non-"..." entries |
| 1293 | BOMB("suppression must contain at least one location " |
| 1294 | "line which is not \"...\""); |
| 1295 | } |
| 1296 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1297 | // Copy tmp_callers[] into supp->callers[] |
| 1298 | supp->n_callers = i; |
sewardj | 9c606bd | 2008-09-18 18:12:50 +0000 | [diff] [blame] | 1299 | supp->callers = VG_(arena_malloc)(VG_AR_CORE, "errormgr.losf.4", |
| 1300 | i*sizeof(SuppLoc)); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1301 | for (i = 0; i < supp->n_callers; i++) { |
| 1302 | supp->callers[i] = tmp_callers[i]; |
sewardj | 57a8f5f | 2003-07-06 01:40:11 +0000 | [diff] [blame] | 1303 | } |
| 1304 | |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 1305 | supp->next = suppressions; |
| 1306 | suppressions = supp; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1307 | } |
njn | 35db56c | 2009-07-24 07:38:29 +0000 | [diff] [blame] | 1308 | VG_(free)(buf); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1309 | VG_(close)(fd); |
| 1310 | return; |
| 1311 | |
| 1312 | syntax_error: |
sewardj | f349d55 | 2005-11-14 17:01:01 +0000 | [diff] [blame] | 1313 | if (VG_(clo_xml)) |
sewardj | 09361bf | 2011-01-28 00:19:25 +0000 | [diff] [blame] | 1314 | VG_(printf_xml)("</valgrindoutput>\n"); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1315 | VG_(umsg)("FATAL: in suppressions file \"%s\" near line %d:\n", |
njn | 6f74a7e | 2009-03-12 00:06:45 +0000 | [diff] [blame] | 1316 | filename, lineno ); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1317 | VG_(umsg)(" %s\n", err_str ); |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1318 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1319 | VG_(close)(fd); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1320 | VG_(umsg)("exiting now.\n"); |
nethercote | 8ed8a89 | 2004-11-08 13:24:25 +0000 | [diff] [blame] | 1321 | VG_(exit)(1); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1322 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1323 | # undef BOMB |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1324 | } |
| 1325 | |
| 1326 | |
| 1327 | void VG_(load_suppressions) ( void ) |
| 1328 | { |
| 1329 | Int i; |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 1330 | suppressions = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1331 | for (i = 0; i < VG_(clo_n_suppressions); i++) { |
| 1332 | if (VG_(clo_verbosity) > 1) { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1333 | VG_(dmsg)("Reading suppressions file: %s\n", |
| 1334 | VG_(clo_suppressions)[i] ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1335 | } |
| 1336 | load_one_suppressions_file( VG_(clo_suppressions)[i] ); |
| 1337 | } |
| 1338 | } |
| 1339 | |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1340 | |
| 1341 | /*------------------------------------------------------------*/ |
| 1342 | /*--- Matching errors to suppressions ---*/ |
| 1343 | /*------------------------------------------------------------*/ |
| 1344 | |
| 1345 | /* Parameterising functions for the use of VG_(generic_match) in |
| 1346 | suppression-vs-error matching. The suppression frames (SuppLoc) |
| 1347 | play the role of 'pattern'-element, and the error frames (IPs, |
| 1348 | hence simply Addrs) play the role of 'input'. In short then, we're |
| 1349 | matching a sequence of Addrs against a pattern composed of a |
| 1350 | sequence of SuppLocs. |
| 1351 | */ |
| 1352 | static Bool supploc_IsStar ( void* supplocV ) |
| 1353 | { |
| 1354 | SuppLoc* supploc = (SuppLoc*)supplocV; |
| 1355 | return supploc->ty == DotDotDot; |
| 1356 | } |
| 1357 | |
| 1358 | static Bool supploc_IsQuery ( void* supplocV ) |
| 1359 | { |
| 1360 | return False; /* there's no '?' equivalent in the supp syntax */ |
| 1361 | } |
| 1362 | |
| 1363 | static Bool supp_pattEQinp ( void* supplocV, void* addrV ) |
| 1364 | { |
| 1365 | SuppLoc* supploc = (SuppLoc*)supplocV; /* PATTERN */ |
| 1366 | Addr ip = *(Addr*)addrV; /* INPUT */ |
| 1367 | |
| 1368 | Char caller_name[ERRTXT_LEN]; |
| 1369 | caller_name[0] = 0; |
| 1370 | |
| 1371 | /* So, does this IP address match this suppression-line? */ |
| 1372 | switch (supploc->ty) { |
| 1373 | case DotDotDot: |
| 1374 | /* supp_pattEQinp is a callback from VG_(generic_match). As |
| 1375 | per the spec thereof (see include/pub_tool_seqmatch.h), we |
| 1376 | should never get called with a pattern value for which the |
| 1377 | _IsStar or _IsQuery function would return True. Hence |
| 1378 | this can't happen. */ |
| 1379 | vg_assert(0); |
| 1380 | case ObjName: |
| 1381 | /* Get the object name into 'caller_name', or "???" |
| 1382 | if unknown. */ |
| 1383 | if (!VG_(get_objname)(ip, caller_name, ERRTXT_LEN)) |
| 1384 | VG_(strcpy)(caller_name, "???"); |
| 1385 | break; |
| 1386 | case FunName: |
| 1387 | /* Get the function name into 'caller_name', or "???" |
| 1388 | if unknown. */ |
njn | 6b7611b | 2009-02-11 06:06:10 +0000 | [diff] [blame] | 1389 | // Nb: C++-mangled names are used in suppressions. Do, though, |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1390 | // Z-demangle them, since otherwise it's possible to wind |
| 1391 | // up comparing "malloc" in the suppression against |
| 1392 | // "_vgrZU_libcZdsoZa_malloc" in the backtrace, and the |
| 1393 | // two of them need to be made to match. |
njn | 6b7611b | 2009-02-11 06:06:10 +0000 | [diff] [blame] | 1394 | if (!VG_(get_fnname_no_cxx_demangle)(ip, caller_name, ERRTXT_LEN)) |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1395 | VG_(strcpy)(caller_name, "???"); |
| 1396 | break; |
| 1397 | default: |
| 1398 | vg_assert(0); |
| 1399 | } |
| 1400 | |
| 1401 | /* So now we have the function or object name in caller_name, and |
| 1402 | the pattern (at the character level) to match against is in |
| 1403 | supploc->name. Hence (and leading to a re-entrant call of |
| 1404 | VG_(generic_match)): */ |
| 1405 | return VG_(string_match)(supploc->name, caller_name); |
| 1406 | } |
| 1407 | |
| 1408 | ///////////////////////////////////////////////////// |
| 1409 | |
| 1410 | static Bool supp_matches_callers(Error* err, Supp* su) |
| 1411 | { |
| 1412 | /* Unwrap the args and set up the correct parameterisation of |
| 1413 | VG_(generic_match), using supploc_IsStar, supploc_IsQuery and |
| 1414 | supp_pattEQinp. */ |
| 1415 | /* note, StackTrace === Addr* */ |
| 1416 | StackTrace ips = VG_(get_ExeContext_StackTrace)(err->where); |
| 1417 | UWord n_ips = VG_(get_ExeContext_n_ips)(err->where); |
| 1418 | SuppLoc* supps = su->callers; |
| 1419 | UWord n_supps = su->n_callers; |
| 1420 | UWord szbPatt = sizeof(SuppLoc); |
| 1421 | UWord szbInput = sizeof(Addr); |
| 1422 | Bool matchAll = False; /* we just want to match a prefix */ |
| 1423 | return |
| 1424 | VG_(generic_match)( |
| 1425 | matchAll, |
| 1426 | /*PATT*/supps, szbPatt, n_supps, 0/*initial Ix*/, |
| 1427 | /*INPUT*/ips, szbInput, n_ips, 0/*initial Ix*/, |
| 1428 | supploc_IsStar, supploc_IsQuery, supp_pattEQinp |
| 1429 | ); |
| 1430 | } |
| 1431 | |
| 1432 | ///////////////////////////////////////////////////// |
| 1433 | |
sewardj | b5f6f51 | 2005-03-10 23:59:00 +0000 | [diff] [blame] | 1434 | static |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1435 | Bool supp_matches_error(Supp* su, Error* err) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1436 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1437 | switch (su->skind) { |
njn | a86f29e | 2006-12-14 02:55:58 +0000 | [diff] [blame] | 1438 | //(example code, see comment on CoreSuppKind above) |
| 1439 | //case ThreadSupp: |
| 1440 | // return (err->ekind == ThreadErr); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1441 | default: |
njn | 95ec870 | 2004-11-22 16:46:13 +0000 | [diff] [blame] | 1442 | if (VG_(needs).tool_errors) { |
njn | 51d827b | 2005-05-09 01:02:08 +0000 | [diff] [blame] | 1443 | return VG_TDICT_CALL(tool_error_matches_suppression, err, su); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1444 | } else { |
| 1445 | VG_(printf)( |
njn | 95ec870 | 2004-11-22 16:46:13 +0000 | [diff] [blame] | 1446 | "\nUnhandled suppression type: %u. VG_(needs).tool_errors\n" |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1447 | "probably needs to be set.\n", |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1448 | err->ekind); |
njn | 6799325 | 2004-11-22 18:02:32 +0000 | [diff] [blame] | 1449 | VG_(tool_panic)("unhandled suppression type"); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1450 | } |
| 1451 | } |
| 1452 | } |
| 1453 | |
sewardj | d7a02db | 2008-12-12 08:07:49 +0000 | [diff] [blame] | 1454 | ///////////////////////////////////////////////////// |
sewardj | 5e51930 | 2008-11-03 23:10:25 +0000 | [diff] [blame] | 1455 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1456 | /* Does an error context match a suppression? ie is this a suppressible |
| 1457 | error? If so, return a pointer to the Supp record, otherwise NULL. |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1458 | Tries to minimise the number of symbol searches since they are expensive. |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1459 | */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1460 | static Supp* is_suppressible_error ( Error* err ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1461 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 1462 | Supp* su; |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1463 | Supp* su_prev; |
| 1464 | |
| 1465 | /* stats gathering */ |
| 1466 | em_supplist_searches++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1467 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1468 | /* See if the error context matches any suppression. */ |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1469 | su_prev = NULL; |
njn | 695c16e | 2005-03-27 03:40:28 +0000 | [diff] [blame] | 1470 | for (su = suppressions; su != NULL; su = su->next) { |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1471 | em_supplist_cmps++; |
| 1472 | if (supp_matches_error(su, err) && supp_matches_callers(err, su)) { |
| 1473 | /* got a match. Move this entry to the head of the list |
| 1474 | in the hope of making future searches cheaper. */ |
| 1475 | if (su_prev) { |
| 1476 | vg_assert(su_prev->next == su); |
| 1477 | su_prev->next = su->next; |
| 1478 | su->next = suppressions; |
| 1479 | suppressions = su; |
| 1480 | } |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1481 | return su; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1482 | } |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1483 | su_prev = su; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1484 | } |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1485 | return NULL; /* no matches */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1486 | } |
| 1487 | |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1488 | /* Show accumulated error-list and suppression-list search stats. |
| 1489 | */ |
| 1490 | void VG_(print_errormgr_stats) ( void ) |
| 1491 | { |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1492 | VG_(dmsg)( |
| 1493 | " errormgr: %'lu supplist searches, %'lu comparisons during search\n", |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1494 | em_supplist_searches, em_supplist_cmps |
| 1495 | ); |
sewardj | 738856f | 2009-07-15 14:48:32 +0000 | [diff] [blame] | 1496 | VG_(dmsg)( |
| 1497 | " errormgr: %'lu errlist searches, %'lu comparisons during search\n", |
sewardj | 8a7d4e4 | 2006-10-17 01:41:17 +0000 | [diff] [blame] | 1498 | em_errlist_searches, em_errlist_cmps |
| 1499 | ); |
| 1500 | } |
| 1501 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1502 | /*--------------------------------------------------------------------*/ |
njn | eb8896b | 2005-06-04 20:03:55 +0000 | [diff] [blame] | 1503 | /*--- end ---*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1504 | /*--------------------------------------------------------------------*/ |