Project

General

Profile

Bug #39248

Remove unnecessary pthread_mutex_unlock() in VNC server

Added by Marcelo Araujo about 2 years ago. Updated about 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Marcelo Araujo
Category:
OS
Target version:
Severity:
Low Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

There is a leftover pthread_mutex_unlock() that is not necessary.
Also is safer to call vnc_init_server prior create a thread with vncserver_thr.

Associated revisions

Revision df9a1aa2 (diff)
Added by Marcelo Araujo about 2 years ago

- Remove unnecessary pthread_mutex_unlock(). - Call vnc_init_server prior to create a thread with vncserver_thr. Suggested by: mav@ Ticket: #39248

Revision 10c42376 (diff)
Added by Marcelo Araujo over 1 year ago

- Remove unnecessary pthread_mutex_unlock(). - Call vnc_init_server prior to create a thread with vncserver_thr. Suggested by: mav@ Ticket: #39248 (cherry picked from commit df9a1aa2e36e2d4bf5c58312aa9e92f95f6e8f71)

History

#1 Updated by Marcelo Araujo about 2 years ago

  • Status changed from Unscreened to In Progress
  • Needs Doc changed from Yes to No

#2 Updated by Dru Lavigne about 2 years ago

  • Subject changed from [bhyve] to Remove unnecessary pthread_mutex_unlock() in VNC server

#4 Updated by Dru Lavigne about 2 years ago

  • Status changed from In Progress to Ready for Testing
  • Target version changed from 11.2-BETA3 to 11.2-BETA2
  • Needs Merging changed from Yes to No

#5 Updated by Bonnie Follweiler about 2 years ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

#7 Updated by Dru Lavigne about 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF