[Bug]: if_match parser doesn't handle comparison of strings containing `!` about conky HOT 4 OPEN

Looked into the code, I can confirm it's a bug in the following two functions in algebra.cc:

int find_match_op(const char *expr) {
  unsigned int idx;

  for (idx = 0; idx < strlen(expr); idx++) {
    switch (expr[idx]) {
      case '=':
      case '!':
        if (expr[idx + 1] != '=') { return -1; }
        /* falls through */
      case '<':
      case '>':
        return idx;
    }
  }
  return -1;
}

int get_match_type(const char *expr) {
  int idx;
  const char *str;

  if ((idx = find_match_op(expr)) == -1) { return -1; }
  str = expr + idx;

  if (*str == '=' && *(str + 1) == '=') { return OP_EQ; }
  if (*str == '!' && *(str + 1) == '=') { return OP_NEQ; }
  if (*str == '>') {
    if (*(str + 1) == '=') { return OP_GEQ; }
    return OP_GT;
  }
  if (*str == '<') {
    if (*(str + 1) == '=') { return OP_LEQ; }
    return OP_LT;
  }
  return -1;
}

get_match_type finds the first occurrence of ! via find_match_op and then tries matching it to !=, given that the text can contain arbitrary number of ! before an actual !=, or even the whole != expression, the function get_match_type is faulty because it assumes the comparison is after the first !. FRITZ! routers are very common consumer grade routers provided by ISPs so this was only the most likely failure point.

It should instead parse the first value (string or float), skip the value, skip whitespace, then parse operator, fail if wrong operator or end of text (first string not closed), skip whitespace, parse second value/operand, fail if types not comparable (handle string to number conversion), return comparison result.

Workaround: use exec and get the same information from shell, pipe through sed s/!//g

from conky.

Marking as low priority because #1001 needs to be addressed first (work on #1879). My guess is that this has to do with if_match instead of networking.

Can you provide an example which doesn't use the wireless_essid variable, even if echoing from a file with similar content? Doing so would allow us to unlink the issues.

from conky.

You are right, the same problem exists when using exec:

conky.text = [[${color white}
TEST: ${if_match "${exec echo FRITZ!Box 7520 HI}" == "off/any"}a${else}b$endif
]]

This configuration produces the same error. Is there any special meaning for the exclamation mark in if_match?

from conky.

Thank you for the triage!

I will use the workaround for now. Unfortunately, I don't think I can help in the development for now, but I am available for any test/experimenti :-)

from conky.