Skip to content
Snippets Groups Projects

samba: use the new API

Merged Thomas Guillem requested to merge tguillem/vlc:samba-threadsafe into master
All threads resolved!

And fix potential data races, that could have become criticial with the new medialibary (more than one samba access in //). Indeed, the SMBCCTX and SMBCFILE were stored statically and fetch from an unique fd without any lock mechanism.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thomas Guillem added 39 commits

    added 39 commits

    Compare with previous version

  • Thomas Guillem resolved all threads

    resolved all threads

  • Thomas Guillem added 3 commits

    added 3 commits

    Compare with previous version

  • Thomas Guillem added 3 commits

    added 3 commits

    • eca2d51b - 1 commit from branch videolan:master
    • a80ab727 - samba: use the new API
    • 3ea71c06 - samba: fix talloc issues (leading to abort)

    Compare with previous version

  • Thomas Guillem resolved all threads

    resolved all threads

  • Alexandre Janniaux
  • Thomas Guillem added 1 commit

    added 1 commit

    • b4be4f2b - samba: fix talloc issues (leading to abort)

    Compare with previous version

  • Thomas Guillem added 1 commit

    added 1 commit

    • 7c11b80d - samba: fix talloc issues (leading to abort)

    Compare with previous version

  • Thomas Guillem resolved all threads

    resolved all threads

  • MR Acceptance result

    :tada: This MergeRequest has been Accepted! Congratulations.

    MR acceptance checks details:

    • :white_check_mark: MR should be considered mergeable by Gitlab
    • :white_check_mark: Last pipeline should be successful
    • :white_check_mark: MergeRequest should have at least one external review and/or vote
    • :white_check_mark: All threads should be resolved, and score >= 0
    • :white_check_mark: MergeRequest should have no activity (threads/votes) for (24h/24h)

    This message was automatically generated by homer-bot.

  • Jean-Baptiste Kempf approved this merge request

    approved this merge request

  • added 34 commits

    Compare with previous version

  • Jean-Baptiste Kempf enabled an automatic merge when the pipeline for 2a6a684d succeeds

    enabled an automatic merge when the pipeline for 2a6a684d succeeds

  • Please register or sign in to reply
    Loading