1. 20 Apr, 2021 4 commits
  2. 19 Apr, 2021 7 commits
    • Yukihiro "Matz" Matsumoto's avatar
      Merge pull request #5415 from dearblue/unwind-mrb_protect · ff366b65
      Yukihiro "Matz" Matsumoto authored
      Introducing the `mrb_protect_raw()` API function
      ff366b65
    • dearblue's avatar
      Introducing the `mrb_protect_raw()` API function · 891e8522
      dearblue authored
      The purpose is two-fold:
      
       1. to be able to specify a pointer directly when user data is used
      
          When using `mrb_protect()`, it is necessary to allocate objects by `mrb_obj_cptr()` function when using user data.
      
          Adding `mrb_protect_raw()` will make it simpler to reimplement `mrbgems/mruby-error`.
      
       2. to correctly unwind callinfo when an exception is raised from a C function defined as a method (the main topic)
      
          If a method call is made directly under `mrb_protect()` and a C function is called, control is returned from `mrb_protect()` if an exception occurs there.
          In this case, callinfo is not restored, so it is out of sync.
          Moreover, returning to mruby VM (`mrb_vm_exec()` function) in this state will indicate `ci->pc` of C function which is equal to `NULL`, and subsequent `JUMP` will cause `SIGSEGV`.
      
          Following is an example that actually causes `SIGSEGV`:
      
          - `crash.c`
      
            ```c
            #include <mruby.h>
            #include <mruby/compile.h>
            #include <mruby/error.h>
      
            static mrb_value
            level1_body(mrb_state *mrb, mrb_value self)
            {
              return mrb_funcall(mrb, self, "level2", 0);
            }
      
            static mrb_value
            level1(mrb_state *mrb, mrb_value self)
            {
              return mrb_protect(mrb, level1_body, self, NULL);
            }
      
            static mrb_value
            level2(mrb_state *mrb, mrb_value self)
            {
              mrb_raise(mrb, E_RUNTIME_ERROR, "error!");
              return mrb_nil_value();
            }
      
            int
            main(int argc, char *argv[])
            {
              mrb_state *mrb = mrb_open();
              mrb_define_method(mrb, mrb->object_class, "level1", level1, MRB_ARGS_NONE());
              mrb_define_method(mrb, mrb->object_class, "level2", level2, MRB_ARGS_NONE());
              mrb_p(mrb, mrb_load_string(mrb, "p level1"));
              mrb_close(mrb);
              return 0;
            }
            ```
      
          - compile & run
      
            ```console
            % `bin/mruby-config --cc --cflags --ldflags` crash.c `bin/mruby-config --libs`
      
            % ./a.out
            zsh: segmentation fault (core dumped)  ./a.out
            ```
      
          After applying this patch, it will print exception object and exit normally.
      
      The `mrb_protect()`, `mrb_ensure()` and `mrb_rescue_exceptions()` in `mrbgems/mruby-error` have been rewritten using `mrb_protect_raw()`.
      891e8522
    • Yukihiro "Matz" Matsumoto's avatar
      backtrace.c: remove `MRB_API` from internal functions. · 9d362f2c
      Yukihiro "Matz" Matsumoto authored
      - `mrb_exc_backtrace` to implement `Exception#backtrace`
      - `mrb_get_backtrace` to implement `#caller`
      9d362f2c
    • Yukihiro "Matz" Matsumoto's avatar
      backtrace.c: small refactoring in `mrb_exc_backtrace()`; ref #5394 · 13eba431
      Yukihiro "Matz" Matsumoto authored
      - add comment for unpacking
      - avoid saving the symbol in a local variable
      13eba431
    • Yukihiro "Matz" Matsumoto's avatar
    • Yukihiro "Matz" Matsumoto's avatar
      backtrace.c: should skip first (innermost) frame w/out position; #5394 · 85808e50
      Yukihiro "Matz" Matsumoto authored
      Otherwise we suffer `(unknown):0:` errors.
      85808e50
    • Yukihiro "Matz" Matsumoto's avatar
      time.c: add integer boundary check for year. · 049ec905
      Yukihiro "Matz" Matsumoto authored
      On configurations where `sizeof(mrb_int) > sizeof(int)`.
      049ec905
  3. 17 Apr, 2021 3 commits
  4. 16 Apr, 2021 1 commit
    • John Bampton's avatar
      feat(CI): add the GitHub Super Linter · 9d32d440
      John Bampton authored
      The GitHub Super Linter is a more robust and better supported
      tool than the current GitHub Actions we are using.
      
      Running these checks:
      
      ERROR_ON_MISSING_EXEC_BIT: true
      VALIDATE_BASH: true
      VALIDATE_BASH_EXEC: true
      VALIDATE_EDITORCONFIG: true
      VALIDATE_MARKDOWN: true
      VALIDATE_SHELL_SHFMT: true
      VALIDATE_YAML: true
      
      https://github.com/marketplace/actions/super-linter
      https://github.com/github/super-linter
      
      Added the GitHub Super Linter badge to the README.
      
      Also updated the pre-commit framework and added
      more documentation on pre-commit.
      
      Added one more pre-commit check: check-executables-have-shebangs
      
      Added one extra check for merge conflicts to our
      GitHub Actions.
      
      EditorConfig and Markdown linting.
      
      Minor grammar and spelling fixes.
      
      Update linter.yml
      9d32d440
  5. 15 Apr, 2021 2 commits
  6. 13 Apr, 2021 4 commits
  7. 12 Apr, 2021 5 commits
  8. 10 Apr, 2021 7 commits
  9. 09 Apr, 2021 2 commits
  10. 07 Apr, 2021 5 commits