Summary: | [PATCH] sys-apps/sandbox-1.3.3 memory corruption free(): Invalid argument | ||
---|---|---|---|
Product: | Portage Development | Reporter: | Maximilian Grothusmann <maxi> |
Component: | Sandbox | Assignee: | Sandbox Maintainers <sandbox> |
Status: | RESOLVED FIXED | ||
Severity: | critical | CC: | eaph, jer, marsclic, python, rion4ik, zzam |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
We always need our own strndup()
calloc/realloc/strdup: Don't use xzalloc/xmalloc |
Description
Maximilian Grothusmann
2009-02-01 03:11:04 UTC
Created attachment 180504 [details, diff]
We always need our own strndup()
Remark: Why are you using xcalloc/xmalloc in calloc/realloc/strdup? Isn't that backwards? It makes xcalloc/xrealloc/xstrdup bogus, as those then check for things that can't happen. Created attachment 180505 [details, diff] calloc/realloc/strdup: Don't use xzalloc/xmalloc Attaching a patch regarding comment #2, in case you agree with me. :) better to avoid all the weird overhead in the first place (and thus avoid strndup propagation). thanks for the patches though and tracking this down ... i had noticed there was an issue, but hadnt looked into what was causing it. http://git.overlays.gentoo.org/gitweb/?p=proj/sandbox.git;a=commitdiff;h=6b0d80b98ba7da7facd9b4be901905fe25516d11 Plese, don't close the bug until it's fixed in tree. i track git. fixes will propagate into the tree eventually. the bug in question isnt a big deal and doesnt break any code that isnt already broken. *** Bug 257488 has been marked as a duplicate of this bug. *** *** Bug 257116 has been marked as a duplicate of this bug. *** Are these real duplicates? *** Bug 257701 has been marked as a duplicate of this bug. *** Is there a fix yet for the py object problem itself? How is it related to this sandbox problem? those are not duplicates. this issue is about the display problem and nothing else. stop duping/re-opening. (In reply to comment #12) > stop duping/re-opening. Sorry, but that's completely nonsensical. no it isnt. this bug has nothing to do with .py/.pyc sandbox violations. marking those as dupes of this or re-opening this bug is wrong. Comment on attachment 180505 [details, diff] calloc/realloc/strdup: Don't use xzalloc/xmalloc sorry, i'd forgotten about this one ... ive applied this fix in git now, thanks http://git.overlays.gentoo.org/gitweb/?p=proj/sandbox.git;a=commitdiff;h=541bbacc5a7b5f2f98ce9b64d05b8e3bb94ca211 |