Http.error suite à une mise à jour du core (vers la 4.4.17)

TiTidom · Septembre 12, 2024, 12:02

Bonjour,

Suite à la mise à jour (OS : Debian11) vers la version core 4.4.17 (mais pas sûr que cela vienne explicitement de là, c’est juste là que je l’ai remarqué) ce matin, j’ai vu ces messages dans les logs http.error :

0000|[Thu Sep 12 12:59:08.257407 2024] [php7:notice] [pid 140027:tid 140027] [client 2a01:e0a:3f8:94e1:409b:93ac:2209:d93e:52805] PHP Notice:  Trying to access array offset on value of type null in /var/www/html/core/class/system.class.php on line 419
0001|[Thu Sep 12 12:59:08.257426 2024] [php7:warn] [pid 140027:tid 140027] [client 2a01:e0a:3f8:94e1:409b:93ac:2209:d93e:52805] PHP Warning:  Invalid argument supplied for foreach() in /var/www/html/core/class/system.class.php on line 419

La fameuse ligne 419 correspond dans le code à un case 'yarn' dont le foreach (contrairement aux autres cas) n’est pas protégé sur le contenu de la variable $datas['dependencies']

github.com

jeedom/core/blob/df1b6ceea9ddcde04e355fd50138a2056a2985c6/core/class/system.class.php#L418


      
          							foreach ($value['dependencies'] as $key2 => $value2) {
          								self::$_installPackage[$type_key][mb_strtolower($key2)] = array(
          									'version' => $value2['version']
          								);
          							}
          						}
          					}
          				}
          				break;
          			case 'yarn':
          				$datas = json_decode(shell_exec('cat `' . self::getCmdSudo() . ' yarn global dir`/package.json 2>/dev/null'), true);
          				foreach ($datas['dependencies'] as $key => $value) {
          					self::$_installPackage[$type_key][mb_strtolower($key)] = array(
          						'version' => json_decode(shell_exec('yarn info ' . $key . ' version --json 2>/dev/null'), true)['data']
          					);
          				}
          				break;
          			case 'composer':
          				$datas = json_decode(shell_exec(self::getCmdSudo() . ' composer show -f json 2>/dev/null'), true);
          				foreach ($datas['installed'] as $value) {
          					self::$_installPackage[$type_key][mb_strtolower($value['name'])] = array('version' => $value['version']);

Est-ce qu’il ne faudrait pas ajouter un test du type :

if (isset($datas['dependencies']) && is_array($datas['dependencies']) && count($datas['dependencies']) > 0) {

Comme c’est le cas un peu plus haut à la ligne 402 par exemple, et j’imagine tant qu’à faire, faire la même chose pour les case 'composer' et case 'plugin' un peu plus bas.

Je remonte cela au cas où cela n’ait pas été déjà vu, et si cela a déjà été remonté, alors désolé par avance.

Bonne journée,
TiTidom.

Loic · Septembre 12, 2024, 12:37

Bonjour,
Merci pour le retour je viens de pousser la correction en alpha ca sera donc disponible dans la 4.5

TiTidom · Septembre 12, 2024, 12:41

Merci Loic,

Bonne journée,
TiTidom.

system · Septembre 13, 2024, 12:42

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.