Skip to content

Make ruby workable with CALC_EXACT_MALLOC_SIZE set to 1 #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
funny-falcon wants to merge 10 commits into from
Closed

Make ruby workable with CALC_EXACT_MALLOC_SIZE set to 1 #78

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
65c2d16
fix rb_objspace_free
funny-falcon Jan 6, 2012
09ed965
fix Init_BareVM
funny-falcon Jan 6, 2012
22b380e
fix dln_find
funny-falcon Jan 6, 2012
4c9051d
fix run_exec_dup2
funny-falcon Jan 6, 2012
fa94407
regint - use ruby_xmalloc and ruby_xfree for oniguruma
funny-falcon Jan 6, 2012
9f4bcbe
fix cfunc.c
funny-falcon Jan 6, 2012
083c6eb
fix syslog.c
funny-falcon Jan 6, 2012
982eef3
better way to fix cfunc.c
funny-falcon Jan 8, 2012
ecd6b98
fix readline
funny-falcon Jan 8, 2012
9d97846
fix error in cfunc.c
funny-falcon Jan 9, 2012
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix syslog.c
  • Loading branch information
@funny-falcon
funny-falcon committed Jan 6, 2012
commit 083c6ebc1a23680fb2600ae49a4169fa65dce84d
2 changes: 1 addition & 1 deletion ext/syslog/syslog.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ static VALUE mSyslog_close(VALUE self)

closelog();

free((void *)syslog_ident);
xfree((void *)syslog_ident);
Copy link
Member

@nurse nurse Jan 8, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

syslog_ident is allocated by malloc in strdup, so this should be wrong

Copy link
Author

@funny-falcon funny-falcon Jan 9, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

strdup is overwritten by #include "ruby/util.h" on top of syslog.c

syslog_ident = NULL;
syslog_options = syslog_facility = syslog_mask = -1;
syslog_opened = 0;
Expand Down