microsoft / AirSim

Open source simulator for autonomous vehicles built on Unreal Engine / Unity, from Microsoft AI & Research

Home Page:https://microsoft.github.io/AirSim/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Potential Error world-writable-file-creation Related to CWE-732

Close-Recover opened this issue · comments

Bug report

  • AirSim Version/#commit: latest, but it appears in a much earlier version like v1.7.0-linux
  • UE/Unity version: 4.27
  • N/A
  • OS Version: Linux Ubuntu 18.04

What's the issue you encountered?

Dear developers,

When we use CodeQL (GitHub's own static code analysis tool) to analyze the project, it has reported an error from code that may cause breakdown. And we found that this potential error tends to exist in the project for a relatively long time. In this issue we will provide the error form a single file MavLinkCom/MavLinkTest/Commands.h, including its name and location. Hopefully it will get your attention, and we are looking forward to further communication.

Settings

We use default settings.

How can the issue be reproduced?

  1. Prepare everything by following the guidance of official docs of Build AirSim on Linux before running './build.sh'
  2. Then use CodeQL create database command to establish database and set '--command = './build.sh'', it looks like: codeql database create new-database --language=<language> --command='./build.sh'
  3. Please note that the process needs a clean build and may take a long time to finish. Using clean command and parallelization techniques based on your hardware environment may help you make it faster

Include full error message in text form

  • Because of the limit of GitHub, we cannot attach the original file directly. You can contact us by email to obtain it: 2654209843@qq.com
  • Here is its information that was reported as the most important, it is focused on world-writable-file-creation: Creating a file that is world-writable can allow an attacker to write to the file, which is related to CWE-732:

image

What's better than filing an issue? Filing a pull request :).