Add gst_video_sync integration test#408
Conversation
![sourcelevel-bot[bot]](https://avatars.githubusercontent.com/in/597?s=60&v=4){kind=small}
| rospy.sleep(1) | ||
|
|
||
| with self.assertRaises(subprocess.CalledProcessError): | ||
| subprocess.check_call([ |
There was a problem hiding this comment.
subprocess call - check for execution of untrusted input.
| rospy.sleep(1) | ||
|
|
||
| with self.assertRaises(subprocess.CalledProcessError): | ||
| subprocess.check_call([ |
There was a problem hiding this comment.
Starting a process with a partial executable path
| self.scene_pub.publish(GenericMessage(type='json', message=json.dumps(scene))) | ||
| rospy.sleep(1) | ||
|
|
||
| subprocess.check_call([ |
There was a problem hiding this comment.
subprocess call - check for execution of untrusted input.
| self.scene_pub.publish(GenericMessage(type='json', message=json.dumps(scene))) | ||
| rospy.sleep(1) | ||
|
|
||
| subprocess.check_call([ |
There was a problem hiding this comment.
Starting a process with a partial executable path
| import unittest | ||
| import rospy | ||
| import json | ||
| import subprocess |
There was a problem hiding this comment.
Consider possible security implications associated with subprocess module.
|
SourceLevel has finished reviewing this Pull Request and has found:
But beware that this branch is 41 commits behind the You can see more details about this review at https://app.sourcelevel.io/github/EndPointCorp/lg_ros_nodes/pulls/408. |
1 participant
No description provided.