added pthread mutex to prevent race condition in non thread safe writ…

[tasniabari]

…e within Simulator_Log

Quality Assurance Checklist

To make reviews more efficient, please make sure the software feature meets the following standards and check everything off that meets the quality check. Once everything has been checked, the assigned reviewers will begin the review process. Edit this description to check off the list.

There are exceptions with all guidelines. As long as your decisions are justified, then you are good! Contact the reviewers or the leads about any exceptions.

Requirements

• [ ] Followed Coding Style Guide
• [x] Code Build checks pass
• [x] No merge conflicts
• [ ] Software feature has test associated with it.
• [ ] Software feature has documentation for it updated.
• [ ] If applicable, does software feature have simulation test associated with it (.json & out.json file)
• [ ] Test provides useful information and uses relevant data to accurately represent BPS
  • NOTE: If test file already exists, use that one
• [ ] 2 Members of the team have already reviewed my PR
  • NOTE: Approver will not look at PR if 2 other members haven't reviewed first
• [ ] Did you Have Fun?

Things to Consider

• [ ] Even if the above are checked, is this the best way of writing my code?
  • It's possible to write code that works, but are there ways to make code more efficient?

[manthanand]

minor change make pthread_mutex_t SimulatorLog_Sem; static if its only used in this file but this looks good. Im pretty confident this makes write threadsafe

This is a good comment. It is also something that should be changed (even if it would work otherwise) so next time leave it as a comment and don't approve the PR.