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.

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

Jump to bottom

Fix file descriptor leak, clobbered route updates #652

Merged
merged 3 commits into from
May 24, 2023
Merged

Fix file descriptor leak, clobbered route updates #652

merged 3 commits into from
May 24, 2023

Commits on May 7, 2023

  1. Close file descriptor. 

    Signed-off-by: Tom Carroll <[email protected]>
    @tomc797
    tomc797 committed May 7, 2023
    Configuration menu
    Copy the full SHA
    ec92a4d View commit details
    Browse the repository at this point in the history

  2. Update file offset. 

    Fixes an issue where all the writes occurred at the beginning of the
file.

Signed-off-by: Tom Carroll <[email protected]>
    @tomc797
    tomc797 committed May 7, 2023
    Configuration menu
    Copy the full SHA
    0f4cbd7 View commit details
    Browse the repository at this point in the history

  3. Verify write status 

    If write reports an error or short count, do not execute the route
update command.

Signed-off-by: Tom Carroll <[email protected]>
    @tomc797
    tomc797 committed May 7, 2023
    Configuration menu
    Copy the full SHA
    2ed66c8 View commit details
    Browse the repository at this point in the history