diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2019-10-10 15:45:16 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2019-10-10 20:46:12 +0900 |
| commit | 78e54904b50635c9d45129569c7901c85f1b6883 (patch) | |
| tree | cb6c65f453bda5b5fb31760a99afd43a1f451369 | |
| parent | 8fe6a1904cbbbb0ded1834d174aafdccfef61492 (diff) | |
| download | mruby-78e54904b50635c9d45129569c7901c85f1b6883.tar.gz mruby-78e54904b50635c9d45129569c7901c85f1b6883.zip | |
Silence double free warnings by `mrb_local_free()`.
The warnings were detected by cppcheck.
| -rw-r--r-- | mrbgems/mruby-io/src/file.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/mrbgems/mruby-io/src/file.c b/mrbgems/mruby-io/src/file.c index 2fbda90af..f9633535c 100644 --- a/mrbgems/mruby-io/src/file.c +++ b/mrbgems/mruby-io/src/file.c @@ -147,6 +147,7 @@ mrb_file_s_rename(mrb_state *mrb, mrb_value obj) mrb_locale_free(src); mrb_locale_free(dst); mrb_sys_fail(mrb, RSTRING_CSTR(mrb, mrb_format(mrb, "(%v, %v)", from, to))); + return mrb_fixnum_value(-1); /* not reached */ } mrb_locale_free(src); mrb_locale_free(dst); @@ -253,6 +254,7 @@ mrb_file_realpath(mrb_state *mrb, mrb_value klass) if (realpath(cpath, RSTRING_PTR(result)) == NULL) { mrb_locale_free(cpath); mrb_sys_fail(mrb, cpath); + return result; /* not reached */ } mrb_locale_free(cpath); mrb_str_resize(mrb, result, strlen(RSTRING_PTR(result))); |