KOBAYASHI Shuji
acad9567c1
Unify type of line number to uint16_t
2019-07-15 23:03:41 +09:00
Yukihiro "Matz" Matsumoto
2871d0cdc5
Avoid keeping pointers from mrb_sym2name_len(); fix #4342
...
The addresses for packed inline symbols reference `mrb->symbuf` that
could be overridden by the later call of `mrb_sym2name_len`. Since
file names in call stack information are kept as symbols, keeping the
address in the C structures could cause problems like #4342 .
This changes small incompatible changes in function prototypes:
* `mrb_parser_get_filename`: return value changed to `mrb_sym`.
* `mrb_debug_get_filename`: add `mrb_state*` as a first argument.
* `mrb_debug_get_line`: ditto.
I believe above functions are almost internal, and no third-party
mrbgem use them.
2019-04-01 14:13:06 +09:00
KOBAYASHI Shuji
58ca4061b5
Remove unused variable in each_backtrace()
2019-03-27 19:27:00 +09:00
KOBAYASHI Shuji
c49a10f9a0
Refactor src/backtrace.c
...
- Move calling `mrb_debug_get_filename()` to after `lineno` check.
- Remove unneeded array check in `print_backtrace()`.
- Add a few `const` qualifier.
2019-02-25 20:39:20 +09:00
Yukihiro "Matz" Matsumoto
e986dfa8aa
Merge pull request #4283 from shuujii/reorder-members-in-struct-backtrace_location
...
Reorder members in `struct backtrace_location`
2019-02-18 17:24:48 +09:00
KOBAYASHI Shuji
a28d714398
Reorder members in struct backtrace_location
...
`sizeof(struct backtrace_location)` is 24 bytes -> 16 bytes in LP64
data model etc.
2019-02-18 16:38:56 +09:00
KOBAYASHI Shuji
15344dfd0b
Remove unneeded memset() in src/backtrace.c
2019-02-18 16:12:12 +09:00
Takeshi Watanabe
25c40c2f12
Fix undefined error of pointer dereference.
2018-04-23 12:06:13 +09:00
Yukihiro "Matz" Matsumoto
b7c3a876f1
Add most recent call last message to the backtrace.
2017-11-29 16:56:10 +09:00
Yukihiro "Matz" Matsumoto
e7fe6ee263
Avoid pointer arithmetic in backtrace.c; #3816
2017-11-04 00:35:50 +09:00
Tomoyuki Sahara
f6896751b4
don't overwrite backtrace info.
2017-11-02 15:44:05 +09:00
Tomasz Dąbrowski
4146b5c2b5
fix: src\backtrace.c(83): warning C4244: '=': conversion from 'mrb_int' to 'int', possible loss of data
2017-09-27 22:22:04 +02:00
Yukihiro "Matz" Matsumoto
8bf492f127
Reduce integer type mismatch warnings in VC.
2017-08-12 09:35:35 +09:00
Yukihiro "Matz" Matsumoto
96d4c2d385
Remove an unused argument from each_backtrace_func.
2017-08-11 13:52:24 +09:00
Yukihiro "Matz" Matsumoto
4e2f4f4280
Avoid possible data loss by using ptrdiff_t.
2017-08-11 12:12:11 +09:00
Yukihiro "Matz" Matsumoto
e910cf6169
Check if ptr is NULL before calling memset; fix #3765
2017-08-08 01:54:43 +09:00
Yukihiro "Matz" Matsumoto
718a9ceb6e
Reduce allocation size of backtrace arrays.
2017-07-06 09:33:53 +09:00
Yukihiro "Matz" Matsumoto
542f0f7b16
Avoid out-of-bounds access of the backtrace array.
2017-07-06 09:32:28 +09:00
Yukihiro "Matz" Matsumoto
ab85d3c814
Make break in lambdas work as return according to CRuby behavior.
2017-06-02 14:22:18 +09:00
Yukihiro "Matz" Matsumoto
90fb4dd5b3
Print most recent call last as CPython does.
2017-06-01 09:48:04 +09:00
Yukihiro "Matz" Matsumoto
ff7df939ba
Restore MRB_API function mrb_exc_backtrace(mrb, exc); ref 9644ad5
2017-05-31 23:15:17 +09:00
Yukihiro "Matz" Matsumoto
54b45f6a1b
Use RData instead of String to represent backtrace.
2017-05-26 22:30:27 +09:00
Yukihiro "Matz" Matsumoto
e969229b77
Check backtrace info may not be set (esp. NoMemError); fix #3663
2017-05-25 12:56:19 +09:00
Yukihiro "Matz" Matsumoto
9644ad51b4
Simplify backtrace mechanism; fix #3633 #3634 #3644
...
Instead of preserving a backtrace in `mrb_state`, `mrb_exc_set`
keeps packed backtrace in an exception object. `#backtrace` unpacks
it to an array of strings.
2017-05-23 23:50:42 +09:00
Yukihiro "Matz" Matsumoto
d2458e66c2
Remove class info from backtrace lines.
2017-05-23 23:50:42 +09:00
Yukihiro "Matz" Matsumoto
24a24a41dd
Type check before print_backtrace()
2017-05-23 23:50:42 +09:00
Craig Lehmann
3de2da409b
Avoid call to_s during backtrace traversal
2017-05-15 20:40:47 -04:00
Yukihiro "Matz" Matsumoto
88cd807379
Avoid use of snprintf() when DISABLE_STDIO is set; fix #3632
...
ref #3492 #3515 #3517
2017-04-25 10:41:50 +09:00
Yukihiro "Matz" Matsumoto
77331d127b
Unify else clause style
2017-04-03 16:56:27 +09:00
ksss
b1c5c3b0a5
Fix lost lineno
2017-03-25 15:56:27 +09:00
Yukihiro "Matz" Matsumoto
dcf6a413ca
Use snprintf() to stringify fixnum numbers; fix #3492
2017-03-20 01:39:33 +09:00
Yukihiro "Matz" Matsumoto
c789acbc36
Save/restore arena index in the loop.
2017-03-04 18:52:51 +09:00
Yukihiro "Matz" Matsumoto
8efa7b00df
Preallocate SystemStackError; ref #3421
2017-02-15 11:59:47 +09:00
Yukihiro "Matz" Matsumoto
4b32e651a5
Check if irep is NULL before print_backtrace()
...
According to the valgrind log attached to #3438 , proc->body.irep
may be NULL in some cases.
2017-02-08 23:10:25 +09:00
Yukihiro "Matz" Matsumoto
28b7b9ec64
Skip non string values in backtraces; ref #3408
2017-01-23 16:22:55 +09:00
Tomasz Dąbrowski
1af9e363f2
Fixes for compiling mruby as C++
2016-11-24 09:28:00 +09:00
Yukihiro "Matz" Matsumoto
0c28c7d754
change backtrace sep from const char* to char
2016-03-07 10:59:34 +09:00
Kouhei Sutou
d77c72dac5
Revert "Merge pull request #3126 from jbreeden/backtrace_irep_null_check"
...
This reverts commit bf7719fe8d , reversing
changes made to 4f4fa0ade0 .
We should get backtrace while irep is alive.
2016-03-06 11:51:04 +09:00
jbreeden
d2a8f7e0cc
Fix Travis CI
2016-03-05 14:22:31 -08:00
jbreeden
e02ff850fe
Null check for irep & initialize loc.lineno
2016-03-05 13:45:56 -08:00
Carlo Prelz
d6cbe1bfc6
The original code crashed when mrb->backtrace.n grew to 16.
...
It looks like the logic to reallocate the backtrace was flawed,
based on the wrong variable (loc_raw->i, which, as I have verified,
decreases from 16 to 0 instead of increasing)
I am not sure if this is the correct fix
2016-02-26 09:21:33 +01:00
Simon Génier
146a9eeb38
Fix segfault on mrb_exc_backtrace.
...
The code to iterate over backtrace locations was changed in #3065 , but
unfortunately output_backtrace was not correctly updated to forward the
callback.
2016-01-07 11:24:16 -05:00
Syohei YOSHIDA
931cc0359a
printf precision parameter must be 'int' type
...
There is a problem when MRB_INT64 is enabled.
2016-01-07 11:06:20 +09:00
Kouhei Sutou
a561bdb25f
Support backtrace after method calls
...
GitHub: fix #2902 , #2917
The current implementation traverses stack to retrieve backtrace. But
stack will be changed when some operations are occurred. It means that
backtrace may be broken after some operations.
This change (1) saves the minimum information to retrieve backtrace when
exception is raised and (2) restores backtrace from the minimum
information when backtrace is needed. It reduces overhead for creating
backtrace Ruby objects.
The space for the minimum information is reused by multiple
exceptions. So memory allocation isn't occurred for each exception.
2015-12-29 20:36:12 +09:00
Yukihiro "Matz" Matsumoto
5c405dea3d
include changed from by quotes ("") to by brackets (<>); close #3032
2015-11-27 17:48:23 +09:00
Yukihiro "Matz" Matsumoto
4440566b95
DISABLE_STDIO/ENABLE_DEBUG macros to rename; close #3014
...
changes:
* rename DISABLE_STDIO -> MRB_DISABLE_STDIO
* rename ENABLE_DEBUG -> MRB_ENABLE_DEBUG_HOOK
* no more opposite macro definitions (e.g. ENABLE_STDIO, DISABLE_DEBUG).
* rewrite above macro references throughout the code.
* update documents
2015-11-17 07:30:34 +09:00
Simon Génier
40e4eb599c
DISABLE_STDIO does not disable backtraces.
2015-02-12 13:18:59 -05:00
Simon Génier
df382dea73
Fix possible segfault when accessing backtrace with MRB_WORD_BOXING.
2015-02-05 15:12:21 -05:00
Yukihiro "Matz" Matsumoto
206f89e209
add MRB_API modifiers to mruby API functions
2014-08-04 00:47:08 +09:00
Yukihiro "Matz" Matsumoto
472a1c244e
backtrace supression condition was wrong; fix #2472
2014-07-15 08:34:34 +09:00