mirror/librenms-librenms
1
0
Fork 0
mirror of https://github.com/librenms/librenms.git synced 2024-10-07 16:52:45 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Add error checking for DS name length (#6093)

Browse Source 
* Add error checking for DS name length

* Update RrdDefinition.php

* Move greater than 19 length test to unit test

* Remove unused variables

* Remove duplicate unit test, and update test for empty name

* Typo

* Remove extra assignment

* ...

* Remove unused items

* Update RrdDefinitionTest.php

* Update RrdDefinition.php

* Delete InvalidRrdNameException.php
This commit is contained in:
David Bell
2017-03-09 23:16:35 +00:00
committed by Tony Murray
parent 255d5fb598
commit 00a062c675
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
LibreNMS/RRD/RrdDefinition.php
View File

@ -55,6 +55,10 @@ class RrdDefinition
{
global $config;
if (empty($name)) {
d_echo("DS must be set to a non-empty string.");
}
$ds = array();
$ds[] = $this->escapeName($name);
$ds[] = $this->checkType($type);