Project

General

Profile

Actions
Copy link

Bug #7232

closed

haproxy_pool_edit.php -- sprintf() too few arguments

Added by kevin crawley about 7 years ago. Updated almost 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Renato Botelho
Category:
Web Interface
Target version:
2.4.0
Start date:
02/07/2017
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.4
Affected Architecture:
All

Description

https://github.com/pfsense/FreeBSD-ports/blob/devel/net/pfSense-pkg-haproxy/files/usr/local/www/haproxy/haproxy_pool_edit.php#L695
https://github.com/pfsense/FreeBSD-ports/blob/devel/net/pfSense-pkg-haproxy/files/usr/local/www/haproxy/haproxy_pool_edit.php#L890

unescaped `%` on those lines (two violations on 890)

https://github.com/pfsense/FreeBSD-ports/pull/298

related to https://redmine.pfsense.org/issues/7202

Actions
Copy link
 #1

Updated by Renato Botelho about 7 years ago

  • Status changed from New to Feedback
  • Assignee set to Renato Botelho
  • Target version set to 2.4.0
  • % Done changed from 0 to 100

Fixed in 0.52_5

Actions
Copy link
 #2

Updated by Pi Ba about 7 years ago

Seems like these changes should be reverted?

In webgui im seeing:

CPU to run (around -1%%).
ASCII string of the form 100%%,75%%,drain,down

Is sprintf nolonger used?

Actions
Copy link
 #3

Updated by Renato Botelho about 7 years ago

Reverted

Actions
Copy link
 #4

Updated by Renato Botelho almost 7 years ago

  • Status changed from Feedback to Resolved
Actions
Copy link

Also available in: Atom PDF