On Mon, Jul 2, 2018 at 5:01 AM, Thomas Schmitt <[email protected]> wrote:
> Hi, > > i wrote: > > > Further the previous programmers caused memory leaks with disposal of > > > iso9660_stat_t objects by free(3) rather than iso9660_stat_free(). > > Rocky Bernstein wrote: > > When you find errors like this please just fix them in the main branch. > > I will first have to finish statv2 (and if it's only to demonstrate to > myself that it can be done). > > Is there a way for dummies how to switch between git branches without > endangering uncommitted changes ? > > git stash and git stash pop. > > > In the current implementation this is problematic only with Rock Ridge > > > symbolic links. > > Some of the sins might be harmless because at their position in the code > no Rock Ridge symlinks info would be recorded. > But i think this one is a valid bug: > > http://git.savannah.gnu.org/cgit/libcdio.git/tree/lib/ > iso9660/iso9660_fs.c?h=pbatard-multiextent2&id= > 6ff34038b06ffadb66c23d86cb18f2a460359fae#n1170 > > The freed iso9660_stat_t *p_stat was completely created by > _iso9660_dir_to_statbuf(). So it may indeed have allocated memory at > p_stat->rr.psz_symlink. > > > Have a nice day :) > > Thomas > > >
