On Wed, Sep 13, 2006, Thomas Lotterer wrote: > do not rise privileges for "openpkg rc" when given --eval or --print > options; [...] > - else if ( strcmp(argv[i], "-q") == 0 > + else if ( strcmp(argv[i], "-p") == 0 > + || strcmp(argv[i], "--print") == 0 > + || strcmp(argv[i], "-e") == 0 > + || strcmp(argv[i], "--eval") == 0 > + || strcmp(argv[i], "-q") == 0 > || strcmp(argv[i], "--query") == 0 > || strcmp(argv[i], "-c") == 0 > || strcmp(argv[i], "--config") == 0) {
Err... you are talking about --eval and -print here. That's fine. But you are comparing also for -e (which is "--erase"!) and -p (which is "--package"!) and even if this is what you really want is is done in the section with plain string comparisons while there is (a few lines below the above code) a section which deals which single-char options (and which already handles "-e" -- but there for "--erase" and not "--eval"!). So, I do not think the above is what you really intended. Ralf S. Engelschall [EMAIL PROTECTED] www.engelschall.com ______________________________________________________________________ The OpenPKG Project www.openpkg.org Developer Communication List openpkg-dev@openpkg.org