Ticket #219 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

cli module should strip CRLF

Reported by: erik@… Owned by: lennart
Milestone: 0.9.11 Component: daemon
Keywords: bug? in cli Cc:

Description

Any command in the (tcp-)cli that need an integer, refuse to work. Errors vary from "invalid index" (unload-module) to "Failed to parse volume" (set-sink-volume).

An example of an offending cli command is: set-sink-volume 0 100

This may actually be a bug in strtoul (from the source) as the source seems to be correct (though a bit obfuscated).

Change History

Changed 4 years ago by erik@…

  • priority changed from highest to normal
  • type changed from defect to enhancement

It appears not to be a bug in strtoul() nor a real bug in the pulseaudio daemon/cli-module, the problem is that using the telnet command to connect to the cli, telnet by default adds both linefeed and carriage return to the end of each line. Maybe this can be worked around by filtering out cr's?

Ticket changed to feature request ;-)

Changed 4 years ago by lennart

  • status changed from new to assigned
  • summary changed from Various commands in the cli do not work to cli module should strip CRLF

Changed 4 years ago by lennart

  • status changed from assigned to closed
  • resolution set to fixed

This has been fixed a while back in git.

Changed 4 years ago by lennart

  • milestone set to 0.9.11
Note: See TracTickets for help on using tickets.