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