Skip to content

Unchecked Input for Loop Condition #447

Description

@AlexMiller998s

Risk Rating: Medium

Category Security: Misconfiguration

Description: The application performs some repetitive task in a loop, and defines the number of times to perform the loop according to user input. A very high value could cause the application to get stuck in the loop and to be unable to continue to other operations.

Impact: An attacker could input a very high value, potentially causing a denial of service (DoS).

Remediation:

Recommendation · Ideally, don’t base a loop on user-provided data. If it is necessary to do so, the user input must be first validated and its range should be limited.

Findings:
The following screenshot contains the finding related to this file
https://github.com/videojs/mux.js/blob/v7.1.0/lib/m2ts/metadata-stream.js#L187

Image

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions