Skip to content
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.

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

Running valgrind in a different build in order to detect memory leaks. #31

Merged
merged 11 commits into from
May 21, 2014

Conversation

thedrow
Copy link
Contributor

@thedrow thedrow commented May 21, 2014

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling cd4d464 on thedrow:patch-1 into 5a84e9a on c9s:master.

@c9s
Copy link
Owner

c9s commented May 21, 2014

Looks nice!

@c9s
Copy link
Owner

c9s commented May 21, 2014

But travis ci build failed

@thedrow
Copy link
Contributor Author

thedrow commented May 21, 2014

Because valgrind actually found an error :)

@thedrow
Copy link
Contributor Author

thedrow commented May 21, 2014

Can you pull this so it will be part of the build process?

@c9s
Copy link
Owner

c9s commented May 21, 2014

Hi! can you rebase this PR ? so we may know the key point of the failure.

@thedrow
Copy link
Contributor Author

thedrow commented May 21, 2014

For some reason I can't push. Just merge this and see what happens in the next build.

@c9s
Copy link
Owner

c9s commented May 21, 2014

Well, I guess you need to push + --force

@thedrow
Copy link
Contributor Author

thedrow commented May 21, 2014

I did.

@c9s
Copy link
Owner

c9s commented May 21, 2014

ok it's building https://travis-ci.org/c9s/r3/builds/25682078

@c9s
Copy link
Owner

c9s commented May 21, 2014

since like there is a missing config for it?

./tests/.libs/check_slug: error while loading shared libraries: libr3.so.0: cannot open shared object file: No such file or directory

@c9s
Copy link
Owner

c9s commented May 21, 2014

https://travis-ci.org/c9s/r3/jobs/25683096 there is no leaks now, but it's failed.

@thedrow
Copy link
Contributor Author

thedrow commented May 21, 2014

There is a error: ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 2 from 2)

@thedrow
Copy link
Contributor Author

thedrow commented May 21, 2014

See?

./tests/.libs/check_slug: error while loading shared libraries: libr3.so.0: cannot open shared object file: No such file or directory

==10605== Jump to the invalid address stated on the next line

==10605== at 0x4C6: ???

==10605== by 0x400F04A: _dl_signal_error (dl-error.c:122)

==10605== by 0x400E62A: _dl_map_object_deps (dl-deps.c:705)

==10605== by 0x400350C: dl_main (rtld.c:1827)

==10605== by 0x4015B23: _dl_sysdep_start (dl-sysdep.c:244)

==10605== by 0x4005364: _dl_start (rtld.c:338)

==10605== by 0x40016B7: ??? (in /lib/x86_64-linux-gnu/ld-2.15.so)

==10605== by 0x7: ???

==10605== by 0x7FEFFFE26: ???

==10605== by 0x7FEFFFE3F: ???

==10605== by 0x7FEFFFE58: ???

==10605== by 0x7FEFFFE74: ???

==10605== Address 0x4c6 is not stack'd, malloc'd or (recently) free'd

==10605==

==10605==

==10605== Process terminating with default action of signal 11 (SIGSEGV)

==10605== Bad permissions for mapped region at address 0x4C6

==10605== at 0x4C6: ???

==10605== by 0x400F04A: _dl_signal_error (dl-error.c:122)

==10605== by 0x400E62A: _dl_map_object_deps (dl-deps.c:705)

==10605== by 0x400350C: dl_main (rtld.c:1827)

==10605== by 0x4015B23: _dl_sysdep_start (dl-sysdep.c:244)

==10605== by 0x4005364: _dl_start (rtld.c:338)

==10605== by 0x40016B7: ??? (in /lib/x86_64-linux-gnu/ld-2.15.so)

==10605== by 0x7: ???

==10605== by 0x7FEFFFE26: ???

==10605== by 0x7FEFFFE3F: ???

==10605== by 0x7FEFFFE58: ???

==10605== by 0x7FEFFFE74: ???

@c9s
Copy link
Owner

c9s commented May 21, 2014

yeah because the shared library: libr3 can't be found. so there is an error about _dl_* functions, can you take a look?

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5dbef0c on thedrow:patch-1 into 79fb3a3 on c9s:master.

@c9s
Copy link
Owner

c9s commented May 21, 2014

Great, let me merge this.

c9s added a commit that referenced this pull request May 21, 2014
Running valgrind in a different build in order to detect memory leaks.
@c9s c9s merged commit ceb6153 into c9s:master May 21, 2014
@thedrow thedrow deleted the patch-1 branch May 21, 2014 11:47
@thedrow thedrow restored the patch-1 branch May 21, 2014 12:03
@thedrow thedrow deleted the patch-1 branch December 27, 2015 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants