Thank you for the information.

Reading the code, it's obvious that a "Division by 0" will happen if $meminfo['MemTotal'] is zero (nice catch), but to be honest, you are probably the only user facing this issue, because we've never received similar complaints.

It is very unlikely that there will be another v5 release. As you noticed, the current code in development branch is for Pi-hole v6.

I will keep this issue open until we release v6, so other users can use your fix, if needed.

Why did you change line 44?
This is not needed to fix the issue and I don't think it will be possible to have less than 4 lines if /proc/meminfo exists.

Why did you change line 44?

Because an empty /proc/meminfo results in an array(1) { [0] = "" }.

Meaning the previous > 0 check was always evaluating to true.

Therefore I made the assumption more specific to contain at-least as many elements as accessed. (So we hit our else branch here setting -1)

The max() is an additional case to avoid err 500 if /proc/meminfo returns bogus data.

Both changes seemed valid to me^

This issue is stale because it has been open 30 days with no activity. Please comment or update this issue or it will be closed in 5 days.