sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1 | |
| 2 | /*--------------------------------------------------------------------*/ |
| 3 | /*--- Management of error messages. vg_errcontext.c ---*/ |
| 4 | /*--------------------------------------------------------------------*/ |
| 5 | |
| 6 | /* |
njn | c953984 | 2002-10-02 13:26:35 +0000 | [diff] [blame] | 7 | This file is part of Valgrind, an extensible x86 protected-mode |
| 8 | emulator for monitoring program execution on x86-Unixes. |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 9 | |
nethercote | bb1c991 | 2004-01-04 16:43:23 +0000 | [diff] [blame] | 10 | Copyright (C) 2000-2004 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 | |
| 31 | #include "vg_include.h" |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 32 | |
| 33 | /*------------------------------------------------------------*/ |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 34 | /*--- Globals ---*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 35 | /*------------------------------------------------------------*/ |
| 36 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 37 | /* The list of error contexts found, both suppressed and unsuppressed. |
| 38 | Initially empty, and grows as errors are detected. */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 39 | static Error* vg_errors = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 40 | |
| 41 | /* The list of suppression directives, as read from the specified |
| 42 | suppressions file. */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 43 | static Supp* vg_suppressions = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 44 | |
| 45 | /* Running count of unsuppressed errors detected. */ |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 46 | static UInt n_errs_found = 0; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 47 | |
| 48 | /* Running count of suppressed errors detected. */ |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 49 | static UInt n_errs_suppressed = 0; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 50 | |
| 51 | /* forwards ... */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 52 | static Supp* is_suppressible_error ( Error* err ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 53 | |
| 54 | |
| 55 | /*------------------------------------------------------------*/ |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 56 | /*--- Error type ---*/ |
| 57 | /*------------------------------------------------------------*/ |
| 58 | |
nethercote | 996901a | 2004-08-03 13:29:09 +0000 | [diff] [blame^] | 59 | /* Note: it is imperative this doesn't overlap with (0..) at all, as tools |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 60 | * effectively extend it by defining their own enums in the (0..) range. */ |
| 61 | typedef |
| 62 | enum { |
| 63 | PThreadErr = -1, // Pthreading error |
| 64 | } |
| 65 | CoreErrorKind; |
| 66 | |
| 67 | /* Errors. Extensible (via the 'extra' field). Tools can use a normal |
| 68 | enum (with element values in the normal range (0..)) for `ekind'. |
| 69 | Functions for getting/setting the tool-relevant fields are in |
| 70 | include/vg_skin.h. |
| 71 | |
| 72 | When errors are found and recorded with VG_(maybe_record_error)(), all |
| 73 | the tool must do is pass in the four parameters; core will |
| 74 | allocate/initialise the error record. |
| 75 | */ |
| 76 | struct _Error { |
| 77 | struct _Error* next; |
| 78 | // NULL if unsuppressed; or ptr to suppression record. |
| 79 | Supp* supp; |
| 80 | Int count; |
| 81 | ThreadId tid; |
| 82 | |
| 83 | // The tool-specific part |
| 84 | ExeContext* where; // Initialised by core |
| 85 | Int ekind; // Used by ALL. Must be in the range (0..) |
| 86 | Addr addr; // Used frequently |
| 87 | Char* string; // Used frequently |
| 88 | void* extra; // For any tool-specific extras |
| 89 | }; |
| 90 | |
| 91 | ExeContext* VG_(get_error_where) ( Error* err ) |
| 92 | { |
| 93 | return err->where; |
| 94 | } |
| 95 | |
| 96 | ErrorKind VG_(get_error_kind) ( Error* err ) |
| 97 | { |
| 98 | return err->ekind; |
| 99 | } |
| 100 | |
| 101 | Addr VG_(get_error_address) ( Error* err ) |
| 102 | { |
| 103 | return err->addr; |
| 104 | } |
| 105 | |
| 106 | Char* VG_(get_error_string) ( Error* err ) |
| 107 | { |
| 108 | return err->string; |
| 109 | } |
| 110 | |
| 111 | void* VG_(get_error_extra) ( Error* err ) |
| 112 | { |
| 113 | return err->extra; |
| 114 | } |
| 115 | |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 116 | UInt VG_(get_n_errs_found)( void ) |
| 117 | { |
| 118 | return n_errs_found; |
| 119 | } |
| 120 | |
nethercote | 4a18490 | 2004-08-02 12:21:09 +0000 | [diff] [blame] | 121 | /*------------------------------------------------------------*/ |
| 122 | /*--- Suppression type ---*/ |
| 123 | /*------------------------------------------------------------*/ |
| 124 | |
| 125 | /* Note: it is imperative this doesn't overlap with (0..) at all, as tools |
| 126 | * effectively extend it by defining their own enums in the (0..) range. */ |
| 127 | typedef |
| 128 | enum { |
| 129 | PThreadSupp = -1, /* Matches PThreadErr */ |
| 130 | } |
| 131 | CoreSuppKind; |
| 132 | |
| 133 | /* For each caller specified for a suppression, record the nature of |
| 134 | the caller name. Not of interest to tools. */ |
| 135 | typedef |
| 136 | enum { |
| 137 | ObjName, /* Name is of an shared object file. */ |
| 138 | FunName /* Name is of a function. */ |
| 139 | } |
| 140 | SuppLocTy; |
| 141 | |
| 142 | /* Suppressions. Tools can get/set tool-relevant parts with functions |
| 143 | declared in include/vg_skin.h. Extensible via the 'extra' field. |
| 144 | Tools can use a normal enum (with element values in the normal range |
| 145 | (0..)) for `skind'. */ |
| 146 | struct _Supp { |
| 147 | struct _Supp* next; |
| 148 | Int count; // The number of times this error has been suppressed. |
| 149 | Char* sname; // The name by which the suppression is referred to. |
| 150 | /* First two (name of fn where err occurs, and immediate caller) |
| 151 | * are mandatory; extra two are optional. */ |
| 152 | SuppLocTy caller_ty[VG_N_SUPP_CALLERS]; |
| 153 | Char* caller [VG_N_SUPP_CALLERS]; |
| 154 | |
| 155 | /* The tool-specific part */ |
| 156 | SuppKind skind; // What kind of suppression. Must use the range (0..). |
| 157 | Char* string; // String -- use is optional. NULL by default. |
| 158 | void* extra; // Anything else -- use is optional. NULL by default. |
| 159 | }; |
| 160 | |
| 161 | SuppKind VG_(get_supp_kind) ( Supp* su ) |
| 162 | { |
| 163 | return su->skind; |
| 164 | } |
| 165 | |
| 166 | Char* VG_(get_supp_string) ( Supp* su ) |
| 167 | { |
| 168 | return su->string; |
| 169 | } |
| 170 | |
| 171 | void* VG_(get_supp_extra) ( Supp* su ) |
| 172 | { |
| 173 | return su->extra; |
| 174 | } |
| 175 | |
| 176 | |
| 177 | void VG_(set_supp_kind) ( Supp* su, SuppKind skind ) |
| 178 | { |
| 179 | su->skind = skind; |
| 180 | } |
| 181 | |
| 182 | void VG_(set_supp_string) ( Supp* su, Char* string ) |
| 183 | { |
| 184 | su->string = string; |
| 185 | } |
| 186 | |
| 187 | void VG_(set_supp_extra) ( Supp* su, void* extra ) |
| 188 | { |
| 189 | su->extra = extra; |
| 190 | } |
| 191 | |
| 192 | |
| 193 | /*------------------------------------------------------------*/ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 194 | /*--- Helper fns ---*/ |
| 195 | /*------------------------------------------------------------*/ |
| 196 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 197 | /* Compare error contexts, to detect duplicates. Note that if they |
| 198 | are otherwise the same, the faulting addrs and associated rwoffsets |
| 199 | are allowed to be different. */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 200 | static Bool eq_Error ( VgRes res, Error* e1, Error* e2 ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 201 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 202 | if (e1->ekind != e2->ekind) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 203 | return False; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 204 | if (!VG_(eq_ExeContext)(res, e1->where, e2->where)) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 205 | return False; |
| 206 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 207 | switch (e1->ekind) { |
sewardj | 4dced35 | 2002-06-04 22:54:20 +0000 | [diff] [blame] | 208 | case PThreadErr: |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 209 | vg_assert(VG_(needs).core_errors); |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 210 | if (e1->string == e2->string) |
sewardj | 4dced35 | 2002-06-04 22:54:20 +0000 | [diff] [blame] | 211 | return True; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 212 | if (0 == VG_(strcmp)(e1->string, e2->string)) |
sewardj | 4dced35 | 2002-06-04 22:54:20 +0000 | [diff] [blame] | 213 | return True; |
| 214 | return False; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 215 | default: |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 216 | if (VG_(needs).skin_errors) |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 217 | return SK_(eq_SkinError)(res, e1, e2); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 218 | else { |
| 219 | VG_(printf)("\nUnhandled error type: %u. VG_(needs).skin_errors\n" |
| 220 | "probably needs to be set.\n", |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 221 | e1->ekind); |
njn | e427a66 | 2002-10-02 11:08:25 +0000 | [diff] [blame] | 222 | VG_(skin_panic)("unhandled error type"); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 223 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 224 | } |
| 225 | } |
| 226 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 227 | static void pp_Error ( Error* err, Bool printCount ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 228 | { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 229 | if (printCount) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 230 | VG_(message)(Vg_UserMsg, "Observed %d times:", err->count ); |
| 231 | if (err->tid > 1) |
| 232 | VG_(message)(Vg_UserMsg, "Thread %d:", err->tid ); |
| 233 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 234 | switch (err->ekind) { |
sewardj | 4dced35 | 2002-06-04 22:54:20 +0000 | [diff] [blame] | 235 | case PThreadErr: |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 236 | vg_assert(VG_(needs).core_errors); |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 237 | VG_(message)(Vg_UserMsg, "%s", err->string ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 238 | VG_(pp_ExeContext)(err->where); |
sewardj | 4dced35 | 2002-06-04 22:54:20 +0000 | [diff] [blame] | 239 | break; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 240 | default: |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 241 | if (VG_(needs).skin_errors) |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 242 | SK_(pp_SkinError)( err ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 243 | else { |
| 244 | VG_(printf)("\nUnhandled error type: %u. VG_(needs).skin_errors\n" |
| 245 | "probably needs to be set?\n", |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 246 | err->ekind); |
njn | e427a66 | 2002-10-02 11:08:25 +0000 | [diff] [blame] | 247 | VG_(skin_panic)("unhandled error type"); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 248 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 249 | } |
| 250 | } |
| 251 | |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 252 | /* Figure out if we want to perform a given action for this error, possibly |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 253 | by asking the user. */ |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 254 | Bool VG_(is_action_requested) ( Char* action, Bool* clo ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 255 | { |
| 256 | Char ch, ch2; |
| 257 | Int res; |
| 258 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 259 | if (*clo == False) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 260 | return False; |
| 261 | |
| 262 | VG_(message)(Vg_UserMsg, ""); |
| 263 | |
| 264 | again: |
| 265 | VG_(printf)( |
| 266 | "==%d== " |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 267 | "---- %s ? --- [Return/N/n/Y/y/C/c] ---- ", |
| 268 | VG_(getpid)(), action |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 269 | ); |
| 270 | |
sewardj | 6024b21 | 2003-07-13 10:54:33 +0000 | [diff] [blame] | 271 | res = VG_(read)(VG_(clo_input_fd), &ch, 1); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 272 | if (res != 1) goto ioerror; |
| 273 | /* res == 1 */ |
| 274 | if (ch == '\n') return False; |
| 275 | if (ch != 'N' && ch != 'n' && ch != 'Y' && ch != 'y' |
| 276 | && ch != 'C' && ch != 'c') goto again; |
| 277 | |
sewardj | 6024b21 | 2003-07-13 10:54:33 +0000 | [diff] [blame] | 278 | res = VG_(read)(VG_(clo_input_fd), &ch2, 1); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 279 | if (res != 1) goto ioerror; |
| 280 | if (ch2 != '\n') goto again; |
| 281 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 282 | /* No, don't want to do action. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 283 | if (ch == 'n' || ch == 'N') return False; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 284 | /* Yes, want to do action. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 285 | if (ch == 'y' || ch == 'Y') return True; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 286 | /* No, don't want to do action, and don't ask again either. */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 287 | vg_assert(ch == 'c' || ch == 'C'); |
| 288 | |
| 289 | ioerror: |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 290 | *clo = False; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 291 | return False; |
| 292 | } |
| 293 | |
| 294 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 295 | /* I've gone all object-oriented... initialisation depends on where the |
| 296 | error comes from: |
| 297 | |
| 298 | - If from generated code (tst == NULL), the %EIP/%EBP values that we |
njn | 3e88418 | 2003-04-15 13:03:23 +0000 | [diff] [blame] | 299 | need in order to attach GDB are picked up out of VG_(baseBlock) rather |
| 300 | than from the thread table (vg_threads in vg_scheduler.c). |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 301 | |
| 302 | - If not from generated code but in response to requests passed back to |
| 303 | the scheduler (tst != NULL), we pick up %EIP/%EBP values from the |
| 304 | stored thread state, not from VG_(baseBlock). |
| 305 | */ |
| 306 | static __inline__ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 307 | void construct_error ( Error* err, ThreadId tid, ErrorKind ekind, Addr a, |
| 308 | Char* s, void* extra, ExeContext* where ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 309 | { |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 310 | sk_assert(tid < VG_N_THREADS); |
| 311 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 312 | /* Core-only parts */ |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 313 | err->next = NULL; |
| 314 | err->supp = NULL; |
| 315 | err->count = 1; |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 316 | err->tid = tid; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 317 | if (NULL == where) |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 318 | err->where = VG_(get_ExeContext)( tid ); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 319 | else |
| 320 | err->where = where; |
njn | 1d6c4bc | 2002-11-21 13:38:08 +0000 | [diff] [blame] | 321 | |
nethercote | 996901a | 2004-08-03 13:29:09 +0000 | [diff] [blame^] | 322 | /* Tool-relevant parts */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 323 | err->ekind = ekind; |
| 324 | err->addr = a; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 325 | err->extra = extra; |
sewardj | a602261 | 2003-07-24 23:50:17 +0000 | [diff] [blame] | 326 | err->string = s; |
| 327 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 328 | /* sanity... */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 329 | vg_assert( tid < VG_N_THREADS ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 330 | } |
| 331 | |
nethercote | 10d481a | 2004-01-25 20:33:53 +0000 | [diff] [blame] | 332 | static void gen_suppression(Error* err) |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 333 | { |
sewardj | 05bcdcb | 2003-05-18 10:05:38 +0000 | [diff] [blame] | 334 | Int i; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 335 | UChar buf[M_VG_ERRTXT]; |
nethercote | 77eba60 | 2003-11-13 17:35:04 +0000 | [diff] [blame] | 336 | Bool main_done = False; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 337 | ExeContext* ec = VG_(get_error_where)(err); |
| 338 | Int stop_at = VG_(clo_backtrace_size); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 339 | |
njn | 633de32 | 2003-05-12 20:40:13 +0000 | [diff] [blame] | 340 | if (stop_at > 4) stop_at = 4; /* At most four names */ |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 341 | vg_assert(stop_at > 0); |
| 342 | |
| 343 | VG_(printf)("{\n"); |
| 344 | VG_(printf)(" <insert a suppression name here>\n"); |
njn | 6a23053 | 2003-07-21 10:38:23 +0000 | [diff] [blame] | 345 | |
| 346 | if (PThreadErr == err->ekind) { |
| 347 | VG_(printf)(" core:PThread\n"); |
| 348 | |
| 349 | } else { |
| 350 | Char* name = SK_(get_error_name)(err); |
| 351 | if (NULL == name) { |
| 352 | VG_(message)(Vg_UserMsg, |
nethercote | 137bc55 | 2003-11-14 17:47:54 +0000 | [diff] [blame] | 353 | "(tool does not allow error to be suppressed)"); |
njn | 6a23053 | 2003-07-21 10:38:23 +0000 | [diff] [blame] | 354 | return; |
| 355 | } |
| 356 | VG_(printf)(" %s:%s\n", VG_(details).name, name); |
| 357 | SK_(print_extra_suppression_info)(err); |
| 358 | } |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 359 | |
| 360 | /* This loop condensed from VG_(mini_stack_dump)() */ |
| 361 | i = 0; |
| 362 | do { |
| 363 | Addr eip = ec->eips[i]; |
nethercote | 77eba60 | 2003-11-13 17:35:04 +0000 | [diff] [blame] | 364 | if (i > 0) eip--; /* point to calling line */ |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 365 | if ( VG_(get_fnname_nodemangle) (eip, buf, M_VG_ERRTXT) ) { |
nethercote | 77eba60 | 2003-11-13 17:35:04 +0000 | [diff] [blame] | 366 | // Stop after "main"; if main() is recursive, stop after last main(). |
| 367 | |
| 368 | if ( ! VG_(clo_show_below_main)) { |
| 369 | if (VG_STREQ(buf, "main")) |
| 370 | main_done = True; |
| 371 | else if (main_done) |
| 372 | break; |
| 373 | } |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 374 | VG_(printf)(" fun:%s\n", buf); |
| 375 | } else if ( VG_(get_objname)(eip, buf, M_VG_ERRTXT) ) { |
| 376 | VG_(printf)(" obj:%s\n", buf); |
| 377 | } else { |
| 378 | VG_(printf)(" ???:??? " |
nethercote | 125c91e | 2004-06-27 12:38:17 +0000 | [diff] [blame] | 379 | "# unknown, suppression will not work, sorry\n"); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 380 | } |
| 381 | i++; |
| 382 | } while (i < stop_at && ec->eips[i] != 0); |
| 383 | |
| 384 | VG_(printf)("}\n"); |
| 385 | } |
| 386 | |
njn | b4aee05 | 2003-04-15 14:09:58 +0000 | [diff] [blame] | 387 | static |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 388 | void do_actions_on_error(Error* err, Bool allow_db_attach) |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 389 | { |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 390 | /* Perhaps we want a debugger attach at this point? */ |
| 391 | if (allow_db_attach && |
| 392 | VG_(is_action_requested)( "Attach to debugger", & VG_(clo_db_attach) )) |
njn | 3e88418 | 2003-04-15 13:03:23 +0000 | [diff] [blame] | 393 | { |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 394 | VG_(printf)("starting debugger\n"); |
| 395 | VG_(start_debugger)( err->tid ); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 396 | } |
| 397 | /* Or maybe we want to generate the error's suppression? */ |
| 398 | if (VG_(is_action_requested)( "Print suppression", |
| 399 | & VG_(clo_gen_suppressions) )) { |
nethercote | 42602b1 | 2004-01-25 19:30:29 +0000 | [diff] [blame] | 400 | gen_suppression(err); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 401 | } |
| 402 | } |
| 403 | |
| 404 | /* Shared between VG_(maybe_record_error)() and VG_(unique_error)(), |
| 405 | just for pretty printing purposes. */ |
| 406 | static Bool is_first_shown_context = True; |
| 407 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 408 | /* Top-level entry point to the error management subsystem. |
| 409 | All detected errors are notified here; this routine decides if/when the |
| 410 | user should see the error. */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 411 | void VG_(maybe_record_error) ( ThreadId tid, |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 412 | ErrorKind ekind, Addr a, Char* s, void* extra ) |
| 413 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 414 | Error err; |
| 415 | Error* p; |
| 416 | Error* p_prev; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 417 | UInt extra_size; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 418 | VgRes exe_res = Vg_MedRes; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 419 | static Bool stopping_message = False; |
| 420 | static Bool slowdown_message = False; |
| 421 | static Int vg_n_errs_shown = 0; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 422 | |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 423 | /* After M_VG_COLLECT_NO_ERRORS_AFTER_SHOWN different errors have |
| 424 | been found, or M_VG_COLLECT_NO_ERRORS_AFTER_FOUND total errors |
| 425 | have been found, just refuse to collect any more. This stops |
| 426 | the burden of the error-management system becoming excessive in |
| 427 | extremely buggy programs, although it does make it pretty |
| 428 | pointless to continue the Valgrind run after this point. */ |
sewardj | 2e43290 | 2002-06-13 20:44:00 +0000 | [diff] [blame] | 429 | if (VG_(clo_error_limit) |
sewardj | 72f98ff | 2002-06-13 17:23:38 +0000 | [diff] [blame] | 430 | && (vg_n_errs_shown >= M_VG_COLLECT_NO_ERRORS_AFTER_SHOWN |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 431 | || n_errs_found >= M_VG_COLLECT_NO_ERRORS_AFTER_FOUND)) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 432 | if (!stopping_message) { |
| 433 | VG_(message)(Vg_UserMsg, ""); |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 434 | |
| 435 | if (vg_n_errs_shown >= M_VG_COLLECT_NO_ERRORS_AFTER_SHOWN) { |
| 436 | VG_(message)(Vg_UserMsg, |
| 437 | "More than %d different errors detected. " |
| 438 | "I'm not reporting any more.", |
| 439 | M_VG_COLLECT_NO_ERRORS_AFTER_SHOWN ); |
| 440 | } else { |
| 441 | VG_(message)(Vg_UserMsg, |
| 442 | "More than %d total errors detected. " |
| 443 | "I'm not reporting any more.", |
| 444 | M_VG_COLLECT_NO_ERRORS_AFTER_FOUND ); |
| 445 | } |
| 446 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 447 | VG_(message)(Vg_UserMsg, |
sewardj | f2537be | 2002-04-24 21:03:47 +0000 | [diff] [blame] | 448 | "Final error counts will be inaccurate. Go fix your program!"); |
sewardj | 72f98ff | 2002-06-13 17:23:38 +0000 | [diff] [blame] | 449 | VG_(message)(Vg_UserMsg, |
sewardj | 2e43290 | 2002-06-13 20:44:00 +0000 | [diff] [blame] | 450 | "Rerun with --error-limit=no to disable this cutoff. Note"); |
sewardj | 72f98ff | 2002-06-13 17:23:38 +0000 | [diff] [blame] | 451 | VG_(message)(Vg_UserMsg, |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 452 | "that errors may occur in your program without prior warning from"); |
sewardj | 72f98ff | 2002-06-13 17:23:38 +0000 | [diff] [blame] | 453 | VG_(message)(Vg_UserMsg, |
| 454 | "Valgrind, because errors are no longer being displayed."); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 455 | VG_(message)(Vg_UserMsg, ""); |
| 456 | stopping_message = True; |
| 457 | } |
| 458 | return; |
| 459 | } |
| 460 | |
| 461 | /* After M_VG_COLLECT_ERRORS_SLOWLY_AFTER different errors have |
| 462 | been found, be much more conservative about collecting new |
| 463 | ones. */ |
| 464 | if (vg_n_errs_shown >= M_VG_COLLECT_ERRORS_SLOWLY_AFTER) { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 465 | exe_res = Vg_LowRes; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 466 | if (!slowdown_message) { |
| 467 | VG_(message)(Vg_UserMsg, ""); |
| 468 | VG_(message)(Vg_UserMsg, |
| 469 | "More than %d errors detected. Subsequent errors", |
| 470 | M_VG_COLLECT_ERRORS_SLOWLY_AFTER); |
| 471 | VG_(message)(Vg_UserMsg, |
| 472 | "will still be recorded, but in less detail than before."); |
| 473 | slowdown_message = True; |
| 474 | } |
| 475 | } |
| 476 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 477 | /* Build ourselves the error */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 478 | construct_error ( &err, tid, ekind, a, s, extra, NULL ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 479 | |
| 480 | /* First, see if we've got an error record matching this one. */ |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 481 | p = vg_errors; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 482 | p_prev = NULL; |
| 483 | while (p != NULL) { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 484 | if (eq_Error(exe_res, p, &err)) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 485 | /* Found it. */ |
| 486 | p->count++; |
| 487 | if (p->supp != NULL) { |
| 488 | /* Deal correctly with suppressed errors. */ |
| 489 | p->supp->count++; |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 490 | n_errs_suppressed++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 491 | } else { |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 492 | n_errs_found++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 493 | } |
| 494 | |
| 495 | /* Move p to the front of the list so that future searches |
| 496 | for it are faster. */ |
| 497 | if (p_prev != NULL) { |
| 498 | vg_assert(p_prev->next == p); |
| 499 | p_prev->next = p->next; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 500 | p->next = vg_errors; |
| 501 | vg_errors = p; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 502 | } |
sewardj | 7ebf7c3 | 2003-07-24 21:29:40 +0000 | [diff] [blame] | 503 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 504 | return; |
| 505 | } |
| 506 | p_prev = p; |
| 507 | p = p->next; |
| 508 | } |
| 509 | |
| 510 | /* Didn't see it. Copy and add. */ |
| 511 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 512 | /* OK, we're really going to collect it. The context is on the stack and |
| 513 | will disappear shortly, so we must copy it. First do the main |
| 514 | (non-`extra') part. |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 515 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 516 | Then SK_(update_extra) can update the `extra' part. This is for when |
| 517 | there are more details to fill in which take time to work out but |
| 518 | don't affect our earlier decision to include the error -- by |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 519 | postponing those details until now, we avoid the extra work in the |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 520 | case where we ignore the error. Ugly. |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 521 | |
| 522 | Then, if there is an `extra' part, copy it too, using the size that |
njn | a70114c | 2003-08-19 16:14:42 +0000 | [diff] [blame] | 523 | SK_(update_extra) returned. Also allow for people using the void* |
| 524 | extra field for a scalar value like an integer. |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 525 | */ |
| 526 | |
| 527 | /* copy main part */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 528 | p = VG_(arena_malloc)(VG_AR_ERRORS, sizeof(Error)); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 529 | *p = err; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 530 | |
njn | 6a23053 | 2003-07-21 10:38:23 +0000 | [diff] [blame] | 531 | /* update `extra', for non-core errors (core ones don't use 'extra') */ |
njn | a70114c | 2003-08-19 16:14:42 +0000 | [diff] [blame] | 532 | if (VG_(needs).skin_errors && PThreadErr != ekind) { |
njn | 6a23053 | 2003-07-21 10:38:23 +0000 | [diff] [blame] | 533 | extra_size = SK_(update_extra)(p); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 534 | |
njn | a70114c | 2003-08-19 16:14:42 +0000 | [diff] [blame] | 535 | /* copy block pointed to by `extra', if there is one */ |
| 536 | if (NULL != p->extra && 0 != extra_size) { |
njn | 6a23053 | 2003-07-21 10:38:23 +0000 | [diff] [blame] | 537 | void* new_extra = VG_(malloc)(extra_size); |
| 538 | VG_(memcpy)(new_extra, p->extra, extra_size); |
| 539 | p->extra = new_extra; |
| 540 | } |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 541 | } |
| 542 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 543 | p->next = vg_errors; |
| 544 | p->supp = is_suppressible_error(&err); |
| 545 | vg_errors = p; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 546 | if (p->supp == NULL) { |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 547 | n_errs_found++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 548 | if (!is_first_shown_context) |
| 549 | VG_(message)(Vg_UserMsg, ""); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 550 | pp_Error(p, False); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 551 | is_first_shown_context = False; |
| 552 | vg_n_errs_shown++; |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 553 | do_actions_on_error(p, /*allow_db_attach*/True); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 554 | } else { |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 555 | n_errs_suppressed++; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 556 | p->supp->count++; |
| 557 | } |
| 558 | } |
| 559 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 560 | /* Second top-level entry point to the error management subsystem, for |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 561 | errors that the tool wants to report immediately, eg. because they're |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 562 | guaranteed to only happen once. This avoids all the recording and |
| 563 | comparing stuff. But they can be suppressed; returns True if it is |
njn | 47363ab | 2003-04-21 13:24:40 +0000 | [diff] [blame] | 564 | suppressed. Bool `print_error' dictates whether to print the error. |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 565 | Bool `count_error' dictates whether to count the error in n_errs_found. |
njn | 47363ab | 2003-04-21 13:24:40 +0000 | [diff] [blame] | 566 | */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 567 | Bool VG_(unique_error) ( ThreadId tid, ErrorKind ekind, Addr a, Char* s, |
njn | 3e88418 | 2003-04-15 13:03:23 +0000 | [diff] [blame] | 568 | void* extra, ExeContext* where, Bool print_error, |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 569 | Bool allow_db_attach, Bool count_error ) |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 570 | { |
| 571 | Error err; |
| 572 | |
| 573 | /* Build ourselves the error */ |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 574 | construct_error ( &err, tid, ekind, a, s, extra, where ); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 575 | |
| 576 | /* Unless it's suppressed, we're going to show it. Don't need to make |
| 577 | a copy, because it's only temporary anyway. |
| 578 | |
| 579 | Then update the `extra' part with SK_(update_extra), because that can |
| 580 | have an affect on whether it's suppressed. Ignore the size return |
| 581 | value of SK_(update_extra), because we're not copying `extra'. */ |
| 582 | (void)SK_(update_extra)(&err); |
| 583 | |
| 584 | if (NULL == is_suppressible_error(&err)) { |
njn | 47363ab | 2003-04-21 13:24:40 +0000 | [diff] [blame] | 585 | if (count_error) |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 586 | n_errs_found++; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 587 | |
| 588 | if (print_error) { |
| 589 | if (!is_first_shown_context) |
| 590 | VG_(message)(Vg_UserMsg, ""); |
| 591 | pp_Error(&err, False); |
| 592 | is_first_shown_context = False; |
| 593 | } |
nethercote | 04d0fbc | 2004-01-26 16:48:06 +0000 | [diff] [blame] | 594 | do_actions_on_error(&err, allow_db_attach); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 595 | |
| 596 | return False; |
| 597 | |
| 598 | } else { |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 599 | n_errs_suppressed++; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 600 | return True; |
| 601 | } |
| 602 | } |
| 603 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 604 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 605 | /*------------------------------------------------------------*/ |
| 606 | /*--- Exported fns ---*/ |
| 607 | /*------------------------------------------------------------*/ |
| 608 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 609 | /* These are called not from generated code but from the scheduler */ |
sewardj | 8c82451 | 2002-04-14 04:16:48 +0000 | [diff] [blame] | 610 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 611 | void VG_(record_pthread_error) ( ThreadId tid, Char* msg ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 612 | { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 613 | if (! VG_(needs).core_errors) return; |
njn | 7271864 | 2003-07-24 08:45:32 +0000 | [diff] [blame] | 614 | VG_(maybe_record_error)( tid, PThreadErr, /*addr*/0, msg, /*extra*/NULL ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 615 | } |
| 616 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 617 | void VG_(show_all_errors) ( void ) |
| 618 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 619 | Int i, n_min; |
| 620 | Int n_err_contexts, n_supp_contexts; |
| 621 | Error *p, *p_min; |
| 622 | Supp *su; |
| 623 | Bool any_supp; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 624 | |
| 625 | if (VG_(clo_verbosity) == 0) |
| 626 | return; |
| 627 | |
| 628 | n_err_contexts = 0; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 629 | for (p = vg_errors; p != NULL; p = p->next) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 630 | if (p->supp == NULL) |
| 631 | n_err_contexts++; |
| 632 | } |
| 633 | |
| 634 | n_supp_contexts = 0; |
| 635 | for (su = vg_suppressions; su != NULL; su = su->next) { |
| 636 | if (su->count > 0) |
| 637 | n_supp_contexts++; |
| 638 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 639 | VG_(message)(Vg_UserMsg, |
| 640 | "ERROR SUMMARY: " |
| 641 | "%d errors from %d contexts (suppressed: %d from %d)", |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 642 | n_errs_found, n_err_contexts, |
| 643 | n_errs_suppressed, n_supp_contexts ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 644 | |
| 645 | if (VG_(clo_verbosity) <= 1) |
| 646 | return; |
| 647 | |
| 648 | /* Print the contexts in order of increasing error count. */ |
| 649 | for (i = 0; i < n_err_contexts; i++) { |
| 650 | n_min = (1 << 30) - 1; |
| 651 | p_min = NULL; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 652 | for (p = vg_errors; p != NULL; p = p->next) { |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 653 | if (p->supp != NULL) continue; |
| 654 | if (p->count < n_min) { |
| 655 | n_min = p->count; |
| 656 | p_min = p; |
| 657 | } |
| 658 | } |
njn | e427a66 | 2002-10-02 11:08:25 +0000 | [diff] [blame] | 659 | if (p_min == NULL) VG_(skin_panic)("show_all_errors()"); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 660 | |
| 661 | VG_(message)(Vg_UserMsg, ""); |
| 662 | VG_(message)(Vg_UserMsg, "%d errors in context %d of %d:", |
| 663 | p_min->count, |
| 664 | i+1, n_err_contexts); |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 665 | pp_Error( p_min, False ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 666 | |
| 667 | if ((i+1 == VG_(clo_dump_error))) { |
sewardj | 1e8cdc9 | 2002-04-18 11:37:52 +0000 | [diff] [blame] | 668 | VG_(translate) ( 0 /* dummy ThreadId; irrelevant due to below NULLs */, |
sewardj | 22854b9 | 2002-11-30 14:00:47 +0000 | [diff] [blame] | 669 | p_min->where->eips[0], NULL, NULL, NULL, NULL ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 670 | } |
| 671 | |
| 672 | p_min->count = 1 << 30; |
| 673 | } |
| 674 | |
| 675 | if (n_supp_contexts > 0) |
| 676 | VG_(message)(Vg_DebugMsg, ""); |
| 677 | any_supp = False; |
| 678 | for (su = vg_suppressions; su != NULL; su = su->next) { |
| 679 | if (su->count > 0) { |
| 680 | any_supp = True; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 681 | VG_(message)(Vg_DebugMsg, "supp: %4d %s", su->count, su->sname); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 682 | } |
| 683 | } |
| 684 | |
| 685 | if (n_err_contexts > 0) { |
| 686 | if (any_supp) |
| 687 | VG_(message)(Vg_UserMsg, ""); |
| 688 | VG_(message)(Vg_UserMsg, |
| 689 | "IN SUMMARY: " |
| 690 | "%d errors from %d contexts (suppressed: %d from %d)", |
nethercote | f2b1148 | 2004-08-02 12:36:01 +0000 | [diff] [blame] | 691 | n_errs_found, n_err_contexts, n_errs_suppressed, |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 692 | n_supp_contexts ); |
| 693 | VG_(message)(Vg_UserMsg, ""); |
| 694 | } |
| 695 | } |
| 696 | |
| 697 | /*------------------------------------------------------------*/ |
| 698 | /*--- Standard suppressions ---*/ |
| 699 | /*------------------------------------------------------------*/ |
| 700 | |
| 701 | /* Get a non-blank, non-comment line of at most nBuf chars from fd. |
| 702 | Skips leading spaces on the line. Return True if EOF was hit instead. |
| 703 | */ |
| 704 | |
| 705 | #define VG_ISSPACE(ch) (((ch)==' ') || ((ch)=='\n') || ((ch)=='\t')) |
| 706 | |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 707 | Bool VG_(get_line) ( Int fd, Char* buf, Int nBuf ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 708 | { |
| 709 | Char ch; |
| 710 | Int n, i; |
| 711 | while (True) { |
| 712 | /* First, read until a non-blank char appears. */ |
| 713 | while (True) { |
| 714 | n = VG_(read)(fd, &ch, 1); |
| 715 | if (n == 1 && !VG_ISSPACE(ch)) break; |
| 716 | if (n == 0) return True; |
| 717 | } |
| 718 | |
| 719 | /* Now, read the line into buf. */ |
| 720 | i = 0; |
| 721 | buf[i++] = ch; buf[i] = 0; |
| 722 | while (True) { |
| 723 | n = VG_(read)(fd, &ch, 1); |
| 724 | if (n == 0) return False; /* the next call will return True */ |
| 725 | if (ch == '\n') break; |
| 726 | if (i > 0 && i == nBuf-1) i--; |
| 727 | buf[i++] = ch; buf[i] = 0; |
| 728 | } |
| 729 | while (i > 1 && VG_ISSPACE(buf[i-1])) { |
| 730 | i--; buf[i] = 0; |
| 731 | }; |
| 732 | |
| 733 | /* VG_(printf)("The line is `%s'\n", buf); */ |
| 734 | /* Ok, we have a line. If a non-comment line, return. |
| 735 | If a comment line, start all over again. */ |
| 736 | if (buf[0] != '#') return False; |
| 737 | } |
| 738 | } |
| 739 | |
| 740 | |
| 741 | /* *p_caller contains the raw name of a caller, supposedly either |
| 742 | fun:some_function_name or |
| 743 | obj:some_object_name. |
| 744 | Set *p_ty accordingly and advance *p_caller over the descriptor |
| 745 | (fun: or obj:) part. |
| 746 | Returns False if failed. |
| 747 | */ |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 748 | static Bool setLocationTy ( Char** p_caller, SuppLocTy* p_ty ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 749 | { |
| 750 | if (VG_(strncmp)(*p_caller, "fun:", 4) == 0) { |
| 751 | (*p_caller) += 4; |
| 752 | *p_ty = FunName; |
| 753 | return True; |
| 754 | } |
| 755 | if (VG_(strncmp)(*p_caller, "obj:", 4) == 0) { |
| 756 | (*p_caller) += 4; |
| 757 | *p_ty = ObjName; |
| 758 | return True; |
| 759 | } |
| 760 | VG_(printf)("location should start with fun: or obj:\n"); |
| 761 | return False; |
| 762 | } |
| 763 | |
| 764 | |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 765 | /* Look for "tool" in a string like "tool1,tool2,tool3" */ |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 766 | static __inline__ |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 767 | Bool tool_name_present(Char *name, Char *names) |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 768 | { |
| 769 | Bool found; |
| 770 | Char *s = NULL; /* Shut gcc up */ |
| 771 | Int len = VG_(strlen)(name); |
| 772 | |
| 773 | found = (NULL != (s = VG_(strstr)(names, name)) && |
| 774 | (s == names || *(s-1) == ',') && |
| 775 | (*(s+len) == ',' || *(s+len) == '\0') |
| 776 | ); |
| 777 | |
| 778 | return found; |
| 779 | } |
| 780 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 781 | /* Read suppressions from the file specified in vg_clo_suppressions |
| 782 | and place them in the suppressions list. If there's any difficulty |
| 783 | doing this, just give up -- there's no point in trying to recover. |
| 784 | */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 785 | static void load_one_suppressions_file ( Char* filename ) |
| 786 | { |
| 787 | # define N_BUF 200 |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 788 | Int fd, i; |
| 789 | Bool eof; |
| 790 | Char buf[N_BUF+1]; |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 791 | Char* tool_names; |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 792 | Char* supp_name; |
| 793 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 794 | fd = VG_(open)( filename, VKI_O_RDONLY, 0 ); |
jsgf | f3c3f1a | 2003-10-14 22:13:28 +0000 | [diff] [blame] | 795 | if (fd < 0) { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 796 | VG_(message)(Vg_UserMsg, "FATAL: can't open suppressions file `%s'", |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 797 | filename ); |
| 798 | VG_(exit)(1); |
| 799 | } |
| 800 | |
| 801 | while (True) { |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 802 | /* Assign and initialise the two suppression halves (core and tool) */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 803 | Supp* supp; |
| 804 | supp = VG_(arena_malloc)(VG_AR_CORE, sizeof(Supp)); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 805 | supp->count = 0; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 806 | for (i = 0; i < VG_N_SUPP_CALLERS; i++) supp->caller[i] = NULL; |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 807 | supp->string = supp->extra = NULL; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 808 | |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 809 | eof = VG_(get_line) ( fd, buf, N_BUF ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 810 | if (eof) break; |
| 811 | |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 812 | if (!VG_STREQ(buf, "{")) goto syntax_error; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 813 | |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 814 | eof = VG_(get_line) ( fd, buf, N_BUF ); |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 815 | if (eof || VG_STREQ(buf, "}")) goto syntax_error; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 816 | supp->sname = VG_(arena_strdup)(VG_AR_CORE, buf); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 817 | |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 818 | eof = VG_(get_line) ( fd, buf, N_BUF ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 819 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 820 | if (eof) goto syntax_error; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 821 | |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 822 | /* Check it has the "skin1,skin2,...:supp" form (look for ':') */ |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 823 | i = 0; |
| 824 | while (True) { |
| 825 | if (buf[i] == ':') break; |
| 826 | if (buf[i] == '\0') goto syntax_error; |
| 827 | i++; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 828 | } |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 829 | buf[i] = '\0'; /* Replace ':', splitting into two strings */ |
| 830 | |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 831 | tool_names = & buf[0]; |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 832 | supp_name = & buf[i+1]; |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 833 | |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 834 | /* Is it a core suppression? */ |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 835 | if (VG_(needs).core_errors && tool_name_present("core", tool_names)) |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 836 | { |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 837 | if (VG_STREQ(supp_name, "PThread")) |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 838 | supp->skind = PThreadSupp; |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 839 | else |
| 840 | goto syntax_error; |
| 841 | } |
| 842 | |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 843 | /* Is it a tool suppression? */ |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 844 | else if (VG_(needs).skin_errors && |
nethercote | 7cc9c23 | 2004-01-21 15:08:04 +0000 | [diff] [blame] | 845 | tool_name_present(VG_(details).name, tool_names)) |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 846 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 847 | if (SK_(recognised_suppression)(supp_name, supp)) |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 848 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 849 | /* Do nothing, function fills in supp->skind */ |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 850 | } else |
| 851 | goto syntax_error; |
| 852 | } |
| 853 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 854 | else { |
njn | c40c3a8 | 2002-10-02 11:02:27 +0000 | [diff] [blame] | 855 | /* Ignore rest of suppression */ |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 856 | while (True) { |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 857 | eof = VG_(get_line) ( fd, buf, N_BUF ); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 858 | if (eof) goto syntax_error; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 859 | if (VG_STREQ(buf, "}")) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 860 | break; |
| 861 | } |
| 862 | continue; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 863 | } |
| 864 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 865 | if (VG_(needs).skin_errors && |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 866 | !SK_(read_extra_suppression_info)(fd, buf, N_BUF, supp)) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 867 | goto syntax_error; |
| 868 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 869 | /* "i > 0" ensures at least one caller read. */ |
njn | 633de32 | 2003-05-12 20:40:13 +0000 | [diff] [blame] | 870 | for (i = 0; i <= VG_N_SUPP_CALLERS; i++) { |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 871 | eof = VG_(get_line) ( fd, buf, N_BUF ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 872 | if (eof) goto syntax_error; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 873 | if (i > 0 && VG_STREQ(buf, "}")) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 874 | break; |
njn | 633de32 | 2003-05-12 20:40:13 +0000 | [diff] [blame] | 875 | if (i == VG_N_SUPP_CALLERS) |
| 876 | break; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 877 | supp->caller[i] = VG_(arena_strdup)(VG_AR_CORE, buf); |
| 878 | if (!setLocationTy(&(supp->caller[i]), &(supp->caller_ty[i]))) |
| 879 | goto syntax_error; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 880 | } |
| 881 | |
sewardj | 57a8f5f | 2003-07-06 01:40:11 +0000 | [diff] [blame] | 882 | /* make sure to grab the '}' if the num callers is >= |
| 883 | VG_N_SUPP_CALLERS */ |
| 884 | if (!VG_STREQ(buf, "}")) { |
| 885 | do { |
| 886 | eof = VG_(get_line) ( fd, buf, N_BUF ); |
| 887 | } while (!eof && !VG_STREQ(buf, "}")); |
| 888 | } |
| 889 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 890 | supp->next = vg_suppressions; |
| 891 | vg_suppressions = supp; |
| 892 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 893 | VG_(close)(fd); |
| 894 | return; |
| 895 | |
| 896 | syntax_error: |
| 897 | if (eof) { |
| 898 | VG_(message)(Vg_UserMsg, |
| 899 | "FATAL: in suppressions file `%s': unexpected EOF", |
| 900 | filename ); |
| 901 | } else { |
| 902 | VG_(message)(Vg_UserMsg, |
njn | 11cc925 | 2002-10-07 14:42:59 +0000 | [diff] [blame] | 903 | "FATAL: in suppressions file: `%s': syntax error on: %s", |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 904 | filename, buf ); |
| 905 | } |
| 906 | VG_(close)(fd); |
| 907 | VG_(message)(Vg_UserMsg, "exiting now."); |
| 908 | VG_(exit)(1); |
| 909 | |
| 910 | # undef N_BUF |
| 911 | } |
| 912 | |
| 913 | |
| 914 | void VG_(load_suppressions) ( void ) |
| 915 | { |
| 916 | Int i; |
| 917 | vg_suppressions = NULL; |
| 918 | for (i = 0; i < VG_(clo_n_suppressions); i++) { |
| 919 | if (VG_(clo_verbosity) > 1) { |
| 920 | VG_(message)(Vg_UserMsg, "Reading suppressions file: %s", |
| 921 | VG_(clo_suppressions)[i] ); |
| 922 | } |
| 923 | load_one_suppressions_file( VG_(clo_suppressions)[i] ); |
| 924 | } |
| 925 | } |
| 926 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 927 | /* Return the name of an erring fn in a way which is useful |
| 928 | for comparing against the contents of a suppressions file. |
| 929 | Doesn't demangle the fn name, because we want to refer to |
| 930 | mangled names in the suppressions file. |
sewardj | 99aac97 | 2002-12-26 01:53:45 +0000 | [diff] [blame] | 931 | */ |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 932 | static void get_objname_fnname ( Addr a, Char* obj_buf, Int n_obj_buf, |
| 933 | Char* fun_buf, Int n_fun_buf ) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 934 | { |
| 935 | (void)VG_(get_objname) ( a, obj_buf, n_obj_buf ); |
| 936 | (void)VG_(get_fnname_nodemangle)( a, fun_buf, n_fun_buf ); |
| 937 | } |
| 938 | |
| 939 | static __inline__ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 940 | Bool supp_matches_error(Supp* su, Error* err) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 941 | { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 942 | switch (su->skind) { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 943 | case PThreadSupp: |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 944 | return (err->ekind == PThreadErr); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 945 | default: |
| 946 | if (VG_(needs).skin_errors) { |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 947 | return SK_(error_matches_suppression)(err, su); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 948 | } else { |
| 949 | VG_(printf)( |
| 950 | "\nUnhandled suppression type: %u. VG_(needs).skin_errors\n" |
| 951 | "probably needs to be set.\n", |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 952 | err->ekind); |
njn | e427a66 | 2002-10-02 11:08:25 +0000 | [diff] [blame] | 953 | VG_(skin_panic)("unhandled suppression type"); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 954 | } |
| 955 | } |
| 956 | } |
| 957 | |
| 958 | static __inline__ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 959 | Bool supp_matches_callers(Supp* su, Char caller_obj[][M_VG_ERRTXT], |
| 960 | Char caller_fun[][M_VG_ERRTXT]) |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 961 | { |
| 962 | Int i; |
| 963 | |
njn | 633de32 | 2003-05-12 20:40:13 +0000 | [diff] [blame] | 964 | for (i = 0; i < VG_N_SUPP_CALLERS && su->caller[i] != NULL; i++) { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 965 | switch (su->caller_ty[i]) { |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 966 | case ObjName: if (VG_(string_match)(su->caller[i], |
| 967 | caller_obj[i])) break; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 968 | return False; |
njn | 4ba5a79 | 2002-09-30 10:23:54 +0000 | [diff] [blame] | 969 | case FunName: if (VG_(string_match)(su->caller[i], |
| 970 | caller_fun[i])) break; |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 971 | return False; |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 972 | default: VG_(skin_panic)("supp_matches_callers"); |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 973 | } |
| 974 | } |
| 975 | |
| 976 | /* If we reach here, it's a match */ |
| 977 | return True; |
| 978 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 979 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 980 | /* Does an error context match a suppression? ie is this a suppressible |
| 981 | error? If so, return a pointer to the Supp record, otherwise NULL. |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 982 | Tries to minimise the number of symbol searches since they are expensive. |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 983 | */ |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 984 | static Supp* is_suppressible_error ( Error* err ) |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 985 | { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 986 | Int i; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 987 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 988 | Char caller_obj[VG_N_SUPP_CALLERS][M_VG_ERRTXT]; |
| 989 | Char caller_fun[VG_N_SUPP_CALLERS][M_VG_ERRTXT]; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 990 | |
njn | 810086f | 2002-11-14 12:42:47 +0000 | [diff] [blame] | 991 | Supp* su; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 992 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 993 | /* get_objname_fnname() writes the function name and object name if |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 994 | it finds them in the debug info. So the strings in the suppression |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 995 | file should match these. |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 996 | */ |
| 997 | |
| 998 | /* Initialise these strs so they are always safe to compare, even |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 999 | if get_objname_fnname doesn't write anything to them. */ |
| 1000 | for (i = 0; i < VG_N_SUPP_CALLERS; i++) |
| 1001 | caller_obj[i][0] = caller_fun[i][0] = 0; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1002 | |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1003 | for (i = 0; i < VG_N_SUPP_CALLERS && i < VG_(clo_backtrace_size); i++) { |
njn | 43c799e | 2003-04-08 00:08:52 +0000 | [diff] [blame] | 1004 | get_objname_fnname ( err->where->eips[i], caller_obj[i], M_VG_ERRTXT, |
| 1005 | caller_fun[i], M_VG_ERRTXT ); |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1006 | } |
| 1007 | |
| 1008 | /* See if the error context matches any suppression. */ |
| 1009 | for (su = vg_suppressions; su != NULL; su = su->next) { |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1010 | if (supp_matches_error(su, err) && |
| 1011 | supp_matches_callers(su, caller_obj, caller_fun)) { |
| 1012 | return su; |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1013 | } |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1014 | } |
njn25 | e49d8e7 | 2002-09-23 09:36:25 +0000 | [diff] [blame] | 1015 | return NULL; /* no matches */ |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1016 | } |
| 1017 | |
sewardj | de4a1d0 | 2002-03-22 01:27:54 +0000 | [diff] [blame] | 1018 | /*--------------------------------------------------------------------*/ |
| 1019 | /*--- end vg_errcontext.c ---*/ |
| 1020 | /*--------------------------------------------------------------------*/ |