minor fixes
authorVictor Kirhenshtein <victor@netxms.org>
Thu, 3 Sep 2015 18:57:00 +0000 (21:57 +0300)
committerVictor Kirhenshtein <victor@netxms.org>
Thu, 3 Sep 2015 18:57:12 +0000 (21:57 +0300)
.gitignore
src/server/core/agent.cpp
src/server/core/dcitem.cpp
src/server/core/session.cpp
tests/test-libnxcc/test-libnxcc [deleted file]

index 34e2ea0..733a21c 100644 (file)
@@ -207,6 +207,7 @@ target
 /stamp-h1
 /status
 /tests/test-libnetxms/test-libnetxms
+/tests/test-libnxcc/test-libnxcc
 /tests/test-libnxdb/test-libnxdb
 /tools/inst.tar.gz
 /webui/eclipse
index edec1bb..b97bac7 100644 (file)
@@ -25,7 +25,6 @@
 /**
  * Externals
  */
-extern Queue g_nodePollerQueue;
 void ProcessTrap(SNMP_PDU *pdu, const InetAddress& srcAddr, int srcPort, SNMP_Transport *pTransport, SNMP_Engine *localEngine, bool isInformRq);
 
 /**
index d6c639c..8792c41 100644 (file)
@@ -853,6 +853,7 @@ bool DCItem::transform(ItemValue &value, time_t nElapsedTime)
          break;
       case DCM_AVERAGE_PER_MINUTE:
          nElapsedTime /= 60;  // Convert to minutes
+         /* no break */
       case DCM_AVERAGE_PER_SECOND:
          // Check elapsed time to prevent divide-by-zero exception
          if (nElapsedTime == 0)
index 784deaa..a247119 100644 (file)
@@ -23,6 +23,7 @@
 #include "nxcore.h"
 #include <netxms_mt.h>
 #include <nxtools.h>
+#include <nxstat.h>
 
 #ifdef _WIN32
 #include <psapi.h>
 #define min(a,b) ((a) < (b) ? (a) : (b))
 #endif
 
-#ifndef S_ISREG
-#define S_ISREG(m)     (((m) & S_IFMT) == S_IFREG)
-#endif
-
 // WARNING! this hack works only for d2i_X509(); be carefull when adding new code
 #ifdef OPENSSL_CONST
 # undef OPENSSL_CONST
@@ -11804,12 +11801,8 @@ void ClientSession::sendLibraryImage(NXCPMessage *request)
                                _sntprintf(absFileName, MAX_PATH, _T("%s%s%s%s"), g_netxmsdDataDir, DDIR_IMAGES, FS_PATH_SEPARATOR, guidText);
                                DbgPrintf(5, _T("sendLibraryImage: guid=%s, absFileName=%s"), guidText, absFileName);
 
-#ifdef _WIN32
-                               struct _stat st;
-#else
-                               struct stat st;
-#endif
-                               if (_tstat(absFileName, &st) == 0 && S_ISREG(st.st_mode))
+                               NX_STAT_STRUCT st;
+                               if ((CALL_STAT(absFileName, &st) == 0) && S_ISREG(st.st_mode))
                                {
                                        rcc = RCC_SUCCESS;
                                }
@@ -12316,7 +12309,7 @@ void ClientSession::listServerFileStore(NXCPMessage *request)
       }
    }
 
-       if (m_dwSystemAccess & SYSTEM_ACCESS_READ_FILES || musicFiles)
+       if ((m_dwSystemAccess & SYSTEM_ACCESS_READ_FILES) || musicFiles)
        {
       _tcscpy(path, g_netxmsdDataDir);
       _tcscat(path, DDIR_FILES);
@@ -12327,11 +12320,7 @@ void ClientSession::listServerFileStore(NXCPMessage *request)
          int pos = (int)_tcslen(path);
 
          struct _tdirent *d;
-   #ifdef _WIN32
-         struct _stat st;
-   #else
-         struct stat st;
-   #endif
+         NX_STAT_STRUCT st;
          UINT32 count = 0, varId = VID_INSTANCE_LIST_BASE;
          while((d = _treaddir(dir)) != NULL)
          {
@@ -12359,7 +12348,7 @@ void ClientSession::listServerFileStore(NXCPMessage *request)
                   }
                }
                nx_strncpy(&path[pos], d->d_name, MAX_PATH - pos);
-               if (_tstat(path, &st) == 0)
+               if (CALL_STAT(path, &st) == 0)
                {
                   if (S_ISREG(st.st_mode))
                   {
diff --git a/tests/test-libnxcc/test-libnxcc b/tests/test-libnxcc/test-libnxcc
deleted file mode 100755 (executable)
index 3944edf..0000000
+++ /dev/null
@@ -1,228 +0,0 @@
-#! /bin/bash
-
-# test-libnxcc - temporary wrapper script for .libs/test-libnxcc
-# Generated by libtool (GNU libtool) 2.4.2 Debian-2.4.2-1.7ubuntu1
-#
-# The test-libnxcc program cannot be directly executed until all the libtool
-# libraries that it depends on are installed.
-#
-# This wrapper script should never be moved out of the build directory.
-# If it is, it will not operate correctly.
-
-# Sed substitution that helps us do robust quoting.  It backslashifies
-# metacharacters that are still active within double-quoted strings.
-sed_quote_subst='s/\([`"$\\]\)/\\\1/g'
-
-# Be Bourne compatible
-if test -n "${ZSH_VERSION+set}" && (emulate sh) >/dev/null 2>&1; then
-  emulate sh
-  NULLCMD=:
-  # Zsh 3.x and 4.x performs word splitting on ${1+"$@"}, which
-  # is contrary to our usage.  Disable this feature.
-  alias -g '${1+"$@"}'='"$@"'
-  setopt NO_GLOB_SUBST
-else
-  case `(set -o) 2>/dev/null` in *posix*) set -o posix;; esac
-fi
-BIN_SH=xpg4; export BIN_SH # for Tru64
-DUALCASE=1; export DUALCASE # for MKS sh
-
-# The HP-UX ksh and POSIX shell print the target directory to stdout
-# if CDPATH is set.
-(unset CDPATH) >/dev/null 2>&1 && unset CDPATH
-
-relink_command="(cd /home/victor/Source/NetXMS/tests/test-libnxcc/; { test -z \"\${LIBRARY_PATH+set}\" || unset LIBRARY_PATH || { LIBRARY_PATH=; export LIBRARY_PATH; }; }; { test -z \"\${COMPILER_PATH+set}\" || unset COMPILER_PATH || { COMPILER_PATH=; export COMPILER_PATH; }; }; { test -z \"\${GCC_EXEC_PREFIX+set}\" || unset GCC_EXEC_PREFIX || { GCC_EXEC_PREFIX=; export GCC_EXEC_PREFIX; }; }; { test -z \"\${LD_RUN_PATH+set}\" || unset LD_RUN_PATH || { LD_RUN_PATH=; export LD_RUN_PATH; }; }; { test -z \"\${LD_LIBRARY_PATH+set}\" || unset LD_LIBRARY_PATH || { LD_LIBRARY_PATH=; export LD_LIBRARY_PATH; }; }; PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/instantclient_12_1; export PATH; g++ -g -O2 -fno-rtti -fno-exceptions -o \$progdir/\$file test_libnxcc-test-libnxcc.o  -L/usr/local/lib ../../src/libnxcc/.libs/libnxcc.so ../../src/libnetxms/.libs/libnetxms.so -lz -lresolv -ldl -lpthread -lcrypto -Wl,-rpath -Wl,/home/victor/Source/NetXMS/src/libnxcc/.libs -Wl,-rpath -Wl,/home/victor/Source/NetXMS/src/libnetxms/.libs -Wl,-rpath -Wl,/opt/stronghold/lib)"
-
-# This environment variable determines our operation mode.
-if test "$libtool_install_magic" = "%%%MAGIC variable%%%"; then
-  # install mode needs the following variables:
-  generated_by_libtool_version='2.4.2'
-  notinst_deplibs=' ../../src/libnxcc/libnxcc.la ../../src/libnetxms/libnetxms.la'
-else
-  # When we are sourced in execute mode, $file and $ECHO are already set.
-  if test "$libtool_execute_magic" != "%%%MAGIC variable%%%"; then
-    file="$0"
-
-# A function that is used when there is no print builtin or printf.
-func_fallback_echo ()
-{
-  eval 'cat <<_LTECHO_EOF
-$1
-_LTECHO_EOF'
-}
-    ECHO="printf %s\\n"
-  fi
-
-# Very basic option parsing. These options are (a) specific to
-# the libtool wrapper, (b) are identical between the wrapper
-# /script/ and the wrapper /executable/ which is used only on
-# windows platforms, and (c) all begin with the string --lt-
-# (application programs are unlikely to have options which match
-# this pattern).
-#
-# There are only two supported options: --lt-debug and
-# --lt-dump-script. There is, deliberately, no --lt-help.
-#
-# The first argument to this parsing function should be the
-# script's ../../libtool value, followed by no.
-lt_option_debug=
-func_parse_lt_options ()
-{
-  lt_script_arg0=$0
-  shift
-  for lt_opt
-  do
-    case "$lt_opt" in
-    --lt-debug) lt_option_debug=1 ;;
-    --lt-dump-script)
-        lt_dump_D=`$ECHO "X$lt_script_arg0" | /bin/sed -e 's/^X//' -e 's%/[^/]*$%%'`
-        test "X$lt_dump_D" = "X$lt_script_arg0" && lt_dump_D=.
-        lt_dump_F=`$ECHO "X$lt_script_arg0" | /bin/sed -e 's/^X//' -e 's%^.*/%%'`
-        cat "$lt_dump_D/$lt_dump_F"
-        exit 0
-      ;;
-    --lt-*)
-        $ECHO "Unrecognized --lt- option: '$lt_opt'" 1>&2
-        exit 1
-      ;;
-    esac
-  done
-
-  # Print the debug banner immediately:
-  if test -n "$lt_option_debug"; then
-    echo "test-libnxcc:test-libnxcc:${LINENO}: libtool wrapper (GNU libtool) 2.4.2 Debian-2.4.2-1.7ubuntu1" 1>&2
-  fi
-}
-
-# Used when --lt-debug. Prints its arguments to stdout
-# (redirection is the responsibility of the caller)
-func_lt_dump_args ()
-{
-  lt_dump_args_N=1;
-  for lt_arg
-  do
-    $ECHO "test-libnxcc:test-libnxcc:${LINENO}: newargv[$lt_dump_args_N]: $lt_arg"
-    lt_dump_args_N=`expr $lt_dump_args_N + 1`
-  done
-}
-
-# Core function for launching the target application
-func_exec_program_core ()
-{
-
-      if test -n "$lt_option_debug"; then
-        $ECHO "test-libnxcc:test-libnxcc:${LINENO}: newargv[0]: $progdir/$program" 1>&2
-        func_lt_dump_args ${1+"$@"} 1>&2
-      fi
-      exec "$progdir/$program" ${1+"$@"}
-
-      $ECHO "$0: cannot exec $program $*" 1>&2
-      exit 1
-}
-
-# A function to encapsulate launching the target application
-# Strips options in the --lt-* namespace from $@ and
-# launches target application with the remaining arguments.
-func_exec_program ()
-{
-  case " $* " in
-  *\ --lt-*)
-    for lt_wr_arg
-    do
-      case $lt_wr_arg in
-      --lt-*) ;;
-      *) set x "$@" "$lt_wr_arg"; shift;;
-      esac
-      shift
-    done ;;
-  esac
-  func_exec_program_core ${1+"$@"}
-}
-
-  # Parse options
-  func_parse_lt_options "$0" ${1+"$@"}
-
-  # Find the directory that this script lives in.
-  thisdir=`$ECHO "$file" | /bin/sed 's%/[^/]*$%%'`
-  test "x$thisdir" = "x$file" && thisdir=.
-
-  # Follow symbolic links until we get to the real thisdir.
-  file=`ls -ld "$file" | /bin/sed -n 's/.*-> //p'`
-  while test -n "$file"; do
-    destdir=`$ECHO "$file" | /bin/sed 's%/[^/]*$%%'`
-
-    # If there was a directory component, then change thisdir.
-    if test "x$destdir" != "x$file"; then
-      case "$destdir" in
-      [\\/]* | [A-Za-z]:[\\/]*) thisdir="$destdir" ;;
-      *) thisdir="$thisdir/$destdir" ;;
-      esac
-    fi
-
-    file=`$ECHO "$file" | /bin/sed 's%^.*/%%'`
-    file=`ls -ld "$thisdir/$file" | /bin/sed -n 's/.*-> //p'`
-  done
-
-  # Usually 'no', except on cygwin/mingw when embedded into
-  # the cwrapper.
-  WRAPPER_SCRIPT_BELONGS_IN_OBJDIR=no
-  if test "$WRAPPER_SCRIPT_BELONGS_IN_OBJDIR" = "yes"; then
-    # special case for '.'
-    if test "$thisdir" = "."; then
-      thisdir=`pwd`
-    fi
-    # remove .libs from thisdir
-    case "$thisdir" in
-    *[\\/].libs ) thisdir=`$ECHO "$thisdir" | /bin/sed 's%[\\/][^\\/]*$%%'` ;;
-    .libs )   thisdir=. ;;
-    esac
-  fi
-
-  # Try to get the absolute directory name.
-  absdir=`cd "$thisdir" && pwd`
-  test -n "$absdir" && thisdir="$absdir"
-
-  program=lt-'test-libnxcc'
-  progdir="$thisdir/.libs"
-
-  if test ! -f "$progdir/$program" ||
-     { file=`ls -1dt "$progdir/$program" "$progdir/../$program" 2>/dev/null | /bin/sed 1q`; \
-       test "X$file" != "X$progdir/$program"; }; then
-
-    file="$$-$program"
-
-    if test ! -d "$progdir"; then
-      mkdir "$progdir"
-    else
-      rm -f "$progdir/$file"
-    fi
-
-    # relink executable if necessary
-    if test -n "$relink_command"; then
-      if relink_command_output=`eval $relink_command 2>&1`; then :
-      else
-       printf %s\n "$relink_command_output" >&2
-       rm -f "$progdir/$file"
-       exit 1
-      fi
-    fi
-
-    mv -f "$progdir/$file" "$progdir/$program" 2>/dev/null ||
-    { rm -f "$progdir/$program";
-      mv -f "$progdir/$file" "$progdir/$program"; }
-    rm -f "$progdir/$file"
-  fi
-
-  if test -f "$progdir/$program"; then
-    if test "$libtool_execute_magic" != "%%%MAGIC variable%%%"; then
-      # Run the actual program with our arguments.
-      func_exec_program ${1+"$@"}
-    fi
-  else
-    # The program doesn't exist.
-    $ECHO "$0: error: \`$progdir/$program' does not exist" 1>&2
-    $ECHO "This script is just a wrapper for $program." 1>&2
-    $ECHO "See the libtool documentation for more information." 1>&2
-    exit 1
-  fi
-fi