Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
lxcfs: fix shared library reload #364
Recenly I've made failures to run the constructor or destructor fatal. Mostly, because we can't guarantee correct functionality if they fail. In fact we can't do anything useful at all (Maybe I can come up with something more useful soon.) In any case, this surfaced a bug where on systems that replace/delete the current workding directory of lxcfs will cause the constructor to crash on reload because the path that getcwd() returns does not exist anymore. Fix this by always escaping to root after the construtor has run but also make it optional in case someone is messing with chroot()s or other things. Signed-off-by: Christian Brauner <email@example.com>
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments.