Project

General

Profile

Actions
Copy link

Bug #14974

closed

Arbitrary command execution in rudder relay API due to missing parameter sanitization

Added by Alexis Mousset over 5 years ago. Updated over 1 year ago.

Status:
Released
Priority:
N/A
Assignee:
Benoît PECCATTE
Category:
Security
Target version:
4.1.24
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

There is no validation of the "classes" parameters which allows passing arbitrary arguments to the executed command, allowing access to a local root account through CFEngine (-f parameter and local policy in /tmp for example).

It requires:

  • having a local access to the root server to get root access locally, or
  • using the root IP to get root access to relays

Subtasks 2 (0 open — 2 closed)

Bug #14976: Remove code execution in rudder relay API - 5.1 version ReleasedAlexis MoussetActions
Bug #14977: Parent ticket breaks empty classes case RejectedActions
Actions
Copy link
 #1

Updated by Alexis Mousset over 5 years ago

  • Status changed from New to In progress
  • Assignee set to Alexis Mousset
Actions
Copy link
 #2

Updated by Alexis Mousset over 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Alexis Mousset to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/1944
Actions
Copy link
 #3

Updated by Rudder Quality Assistant over 5 years ago

  • Assignee changed from Benoît PECCATTE to Alexis Mousset
Actions
Copy link
 #4

Updated by Alexis Mousset over 5 years ago

  • Assignee changed from Alexis Mousset to Benoît PECCATTE
Actions
Copy link
 #5

Updated by Alexis Mousset over 5 years ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #6

Updated by Vincent MEMBRÉ over 5 years ago

  • Name check set to To do
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ over 5 years ago

  • Fix check set to To do
Actions
Copy link
 #8

Updated by Alexis Mousset over 5 years ago

  • Subject changed from Remove code execution in rudder relay API to Remote code execution in rudder relay API due to missing parameter sanitization
Actions
Copy link
 #9

Updated by Alexis Mousset over 5 years ago

  • Subject changed from Remote code execution in rudder relay API due to missing parameter sanitization to Code execution in rudder relay API due to missing parameter sanitization
Actions
Copy link
 #10

Updated by Alexis Mousset over 5 years ago

  • Subject changed from Code execution in rudder relay API due to missing parameter sanitization to Arbitrary command execution in rudder relay API due to missing parameter sanitization
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ over 5 years ago

  • Fix check changed from To do to Error - Fixed
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ over 5 years ago

  • Fix check changed from Error - Fixed to To do
Actions
Copy link
 #13

Updated by Alexis Mousset over 5 years ago

  • Name check changed from To do to Reviewed
Actions
Copy link
 #14

Updated by Benoît PECCATTE over 5 years ago

  • Fix check changed from To do to Checked
Actions
Copy link
 #15

Updated by Vincent MEMBRÉ over 5 years ago

This bug has been fixed in Rudder 4.1.24 and 5.0.12 which were released today.

Actions
Copy link
 #16

Updated by Vincent MEMBRÉ about 4 years ago

  • Status changed from Pending release to Released
Actions
Copy link
 #17

Updated by Alexis Mousset over 1 year ago

  • Private changed from Yes to No
Actions
Copy link

Also available in: Atom PDF