Configurable environment for vmtoolsd.

If a user wants to override(*) an environment variable e.g.
TMPDIR for vmtoolsd, the only choices for the user are:
1. Change system environment variable, that may affect more
than vmtoolsd
2. Change vmtoolsd service startup scripts on Linux.
Some of these methods, especially #2 gets overwritten by
upgrades and user is forced to re-apply the change on every
upgrade of VMTools. Also, #2 is somewhat complex due to
different type of VMTools installations and differences
in Linux distros.

We can't override the environment completely from within
service but we can configure the environment to a large
extent once vmtoolsd comes up and reads tools.conf.

*=> "override" term here applies to setting, modifying and/or
unsetting an environment variable.

This is mainly required for system service vmsvc, but
given that vmusr shares code with vmsvc, we can provide
this functionality for both.

Updated example tools.conf with the new configuration.

Make pollGtk resetable.

A simple poll implementation built on top of GLib, pollGtk can only be
inited once in a program's lifetime.  This works for product but not
testcases.

This patch changes the function static variable "inited" to be file static,
and can be reset this variable in PollGtkExit().

Common header file change not applicable to open-vm-tools.

Address Coverity reported issues in the DnD plugin.
  - uninitialized non-static class members.
  - memory leak.
  - potential NULL pointer dereference.

Common header file change not applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Address some issues reported by the Coverity scan of open-vm-tools.

Fix or annotate issues reported in the Coverity scan of open-vm-tools,
as follows:

(1) Fix NULL pointer de-reference in failure case of
CopyStringListEntry.

(2) Add NULL pointer checks before dereferences in
VixMsg_DeObfuscateNamePassword.

(3) Annotate a false positive in VixMsg_EncodeString.

(4) Annotate and add an ASSERT to a false positive in
StrUtil_GetLongestLineLength.

nicinfo: report real nameservers used when using systemd-resolved

If systemd-resolved is used, report the external DNS server, not the
locally installed one.  This is detected by checking if /etc/resolv.conf
is a link to /run/systemd/resolve/stub-resolv.conf.  In that case,
/run/systemd/resolve/resolv.conf will hold the actual DNS server.  See
https://www.freedesktop.org/software/systemd/man/systemd-resolved.service.html

Common source file change not directly applicable to open-vm-tools.

stop systemd-243 udev complaints #371

Address issues from pull request #371 on github:
- fix substiution variables for systemd-243
- fix permissions of rules file
See https://github.com/vmware/open-vm-tools/pull/371

Suppress a couple of coverity false alarms in FileLoggerOpen().

The stat() system call is used to determine whether to rotate logs.
There is no danger of time-of-check vs. time-of-use because the rotation
decision still holds even under the very-unlikely condition that the existing
log file size changes.

When rotating the logs, the service should not stop when a rename() fails
on an old file.  The process ignores the rename() return code intentionally.
The error condition cannot be logged because the process is already in the
log handling context and would either crash or risk a recursion loop
otherwise.  In addition, writing to stdout/stderr is useless, since the
process is running as a service and the stdout/stderr is reopened on /dev/null.

Therefore, the above two coverity issues are suppressed in the code.

Bump the VMware Tools version to 11.0.5 on vmtools-prod-cpd.

Update bora/public/vm_tools_version.h to set the current tools version
to 11.0.5

TOOLS_VERSION_SLEDGEHAMMER_UPDATE1  --> 11.0.5

Fix issue reported by Coverity scan in deployPkg

rand() should not be used for security-related applications, because
linear congruential algorithms are too easy to break.  Use a compliant
random number generator, such as /dev/random or /dev/urandom on
Unix-like systems, and CNG (Cryptography API: Next Generation) on Windows.

Avoid securing disk device info for ZFS pools (at this time)

ZFS filesystem pools device names can appear as single directory name
at the root directory.  The current logic to locate the
/sys/class/blocks/<device> PCI structure based on typical device name
format will fail for ZFS pools.  As an immediate fix to the SIGSEGV,
avoid attempting to locate the PCI info; keep vmtoolsd running.

Addresses https://github.com/vmware/open-vm-tools/issues/378

Trim trailing whitespace from log params in /etc/tools.conf.

Remove trailing whitespace from pathnames and parameters in
tools.conf.

Common header file change not applicable to open-vm-tools.

A LOG goes variadic change in asyncsocket.c

Fix Coverity reported issue in dndUIX11.cpp

- unsigned_compare: This greater-than-or-equal-to-zero comparison of an
  unsigned value is always true. end >= 0UL.
 Function: GetLastDirName

Remove dead code in dndCommon.c reported by Coverity.

[open-vm-tools Coverity] Fix Coverity scan reported issues in copyPasteUIX11.cpp

- Non-static class member mHGGetListTime is not initialized in this constructor
 solution: add it into init list of constructor

- unsigned_compare: This greater-than-or-equal-to-zero comparison of an
  unsigned value is always true. end >= 0UL.
 Function: GetLastDirName

[open-vm-tools Coverity] Fix sign extension issue reported by Coverity.

- sign_extension: Suspicious implicit sign extension: x with type uint16
  (16 bits, unsigned) is promoted in (x << 16) | y to type int (32 bits,
  signed), then sign-extended to type unsigned long (64 bits, unsigned).
  If (x << 16) | y is greater than 0x7FFFFFFF, the upper bits of the
  result will all be 1.
 File: bora-vmsoft/services/plugins/dndcp/pointer.cpp
 Function: PointerSetPos

Changes to common source files not directly applicable to open-vm-tools.

Correct several uninitialied varialbles reported by Coverity in dnd/cp code.

- uninit_member: Non-static class member mRpc is not initialized in this
  constructor nor in any functions that it calls.
 File: bora/lib/dnd/rpcutil/rpcV3Util.cpp
 Function: RpcV3Util

- uninit_member: Non-static class member field mRpcChanCBList.xdrInSize is
  not initialized in this constructor nor in any functions that it calls.
 File: bora/vmx/tools/dndCPTransportGuestRpc.cpp
 Function: DnDCPTransportGuestRpc

- uninit_member: Non-static class member mGHDnDDropOccurred is not initialized
  in this constructor nor in any functions that it calls.
 File: bora-vmsoft/lib/dndGuestBase/dndUIX11.cpp
 Function: DnDUIX11

- uninit_member: Non-static class member m_main is not initialized in this
  constructor nor in any functions that it calls.
 File: bora-vmsoft/services/plugins/dndcp/copyPasteDnDX11.cpp
 Function: CopyPasteDnDX11

- uninit_member: Non-static class member mToolsAppCtx is not initialized in
  this constructor nor in any functions that it calls.
 File: bora-vmsoft/lib/dndGuest/vmGuestDnDCPMgr.hh
 Function: VMGuestDnDCPMgr

- uninit_member: Non-static class member mRpc is not initialized in this
  constructor nor in any functions that it calls.
 File: bora-vmsoft/lib/dndGuestBase/guestDnDSrc.cc
 Function: GuestDnDSrc

- uninit_member: Non-static class member mMsgSrc is not initialized in this
  constructor nor in any functions that it calls.
 File: bora/lib/dnd/rpcutil/rpcV4Util.cpp
 Function: RpcV4Util

- uninit_member: Non-static class member mIsActive is not initialized in this
  constructor nor in any functions that it calls.
 File: bora-vmsoft/lib/dndGuest/guestCopyPasteDest.cc
 Function: GuestCopyPasteDest

Common header file change not applicable to open-vm-tools.

Address issues from Coverity scan of open-vm-tools.

Fix two issues reported by the Coverity scan of open-vm-tools
and annotate a third:

* Validate siginfo read from a pipe in order to address
a tainted data issue reported in signalSource.c.

* Add a "Fall through" comment to address a missing break
reported in panic.c.

* Annotate an intentional NULL-dereference in guestSDKLog.c.

Fix misc. warnings related to strncpy usage.

For few files bundled in open-vm-tools package, one of the code
scanners reported the following warning.

" Easily used incorrectly; doesn't always \0-terminate or check for
invalid pointers"

Checked the files and modified the code to pass 'sizeof dest - 1'
as the third argument to avoid any warnings.

In one of the VGAUTH library functions, added a check for the input buffer
size and return a valid error.

Common header file change not applicable to open-vm-tools.

Suppress a couple of coverity false alarms in FileLoggerOpen().

The stat() system call is used to determine whether to rotate logs.
There is no danger of time-of-check vs. time-of-use because the rotation
decision still holds even under the very-unlikely condition that the existing
log file size changes.

When rotating the logs, the service should not stop when a rename() fails
on an old file.  The process ignores the rename() return code intentionally.
The error condition cannot be logged because the process is already in the
log handling context and would either crash or risk a recursion loop
otherwise.  In addition, writing to stdout/stderr is useless, since the
process is running as a service and the stdout/stderr is reopened on /dev/null.

Therefore, the above two coverity issues are suppressed in the code.

Common header file change not applicable to open-vm-tools.

Fix misc. warnings in base64.c

At few places in the code, %ld is being used as the format specifier for
printing the variables of size_t datatype. Changed it to %FMTSZu.

stop systemd-243 udev complaints #371

Address issues from pull request #371 on github:
- fix substiution variables for systemd-243
- fix permissions of rules file
See https://github.com/vmware/open-vm-tools/pull/371

Common header file change not applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Common source file changes not directly applicable to open-vm-tools.

Fix 'variable scope can be reduced' warnings.

One of the code quality checkers (Codacy) reported the following
issue for a bunch of files bundled in 'open-vm-tools'.

"""
Fix 'The scope of the variable 'var' can be reduced.'
"""

Checked all applicable files and removed the variable initialization
where it's really not necessary.

This is really not a major issue but it's no harm fixing.

LOG goes variadic in dndCPTransportGuestRpc.cpp

Common header file change not applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Code clean up in common source files.

Fix macros so they work in the world of Log Facility newlining.

The Log Facility no longer accepts appending calls into a single
line.  As such, some macros need to be "fixed up".

Record new tools patch and update versions in vm_tools_version.h.

Back out previous code clean up changes.

Code clean up in common source files.

Fix macros so they work in the world of Log Facility newlining.

The Log Facility no longer accepts appending calls into a single
line.  As such, some macros need to be "fixed up".

Fix a minor issue reported by Codacy code scanner.

One of the code quality checkers (Codacy) reported the following
issue for a bunch of files bundled in 'open-vm-tools'.

"""
Fix 'variable has been re-assigned before it's value has been used'
"""

Checked all applicable files and removed the variable initialization
where it's really not necessary.

This is really not a major issue but it's no harm fixing.

Fix minor NULL check in DataMap_Copy function.

One of the code scanners identified the following issue in
DataMap_Copy function.

"""
Either the condition 'src==NULL' is redundant or there is possible null pointer dereference: src.
"""

Fixed the issue by moving the NULL check to the top of the function.

Common header file change not directly applicable to open-vm-tools.

Variadic LOG macros and fewer trailing newlines

It's (long past) time we started using variadic LOG macros
and stopped requiring a newline at the end of every format
string.  A previous removed the newline requirement recently.

-   LOG(N, ("format string\n", arg1, arg2))
+   LOG(N, "format string", arg1, arg2)

The important parts of this change are buried in macro madness.
The key bit is the LOG_BYNAME macro, which now can be written
to be variadic.

To support both styles simultaneously, this change adds a macro
LOGLEVEL_VARIADIC which switches the definition of LOG_BYNAME to
variadic (e.g. remove extra parens).

Following this change, we can convert files to the variadic version
and set LOGLEVEL_VARIADIC.

Assertion after use fix - lib/file/standalone.c

One of the code scanners caught this one.

Common header file change not applicable to open-vm-tools.

Merge branch 'stable-11.0.x'

update version to 11.0.1

update ChangeLog

update Release Notes

Fix leaks in ListAliases and ListMappedAliases.

The 'record' for each alias was being leaked.

Fix leaks in ListAliases and ListMappedAliases.

The 'record' for each alias was being leaked.

Common header file change not applicable to open-vm-tools.

Common source file change not directly applicable to open-vm-tools.

Backing out previous change.

Common source file changes not directly applicable to open-vm-tools.

Common source file changes not directly applicable to open-vm-tools.

Common source file changes not directly applicable to open-vm-tools.

Common header file changes not directly applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Move /etc/vmware-tools/vm-support to /usr/bin/ in open-vm-tools.

Change scripts/Makefile.am to know about the new location.

This fix requires the following updates to the vendor scripts:
The rules files have "chmod /etc/vmware-tools/vm-support".
The spec files have "mv /etc/vmware-tools/vm-support /usr/bin".

Common source file changes not directly applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Common source file change not directly applicable to open-vm-tools.

Hostinfo: Improve ESXi recognition

Fix AliasSafeRenameFiles() in vgauth/serviceImpl/alias.c

If the alias mapfile was empty, the updated alias file was not properly
saved (renamed).  This issues has been fixed.

Common header file change not directly applicable to open-vm-tools.

lib/file: Close on exec bit tested incorrectly

The test should be on access, not flags.

Common source file change not directly applicable to open-vm-tools.

Add a Coverity annotation to lib/vmGuestLib/vmGuestLib.c

A previous change adding Coverity annotations to vmGuestLib.c
omitted one.  Adding for completeness although it does not actually
affect Coverity results.

Flipped the default value of config option "hgfs.packetSize.large" to TRUE.

Common header file change not applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Fix potential overflow in timeSync reported by Coverity

Issue: Potentially overflowing expression: data->timeSyncPeriod * 1000000U
       is evaluated using 32-bit arithmetic, and then used in a context
       that expects an expression of type uint64.
Fix: explict type conversion.

GitHub Issue #367. Remove references to deprecated G_INLINE_FUNC.

G_INLINE_FUNC was a work-around for compilers that didn't support
static inline.  Change uses of it to static inline.

Common header file change not applicable to open-vm-tools.

sync files from stable-11.0.0

Merge tag 'stable-11.0.0'

update ChangeLog

release notes for 11.0.0

update copyright header for lib/include/vmware/guestrpc/tclodefs.h

update build number for release

Fix Misc. issues in vmtoolsLog.c

* g_malloc0 aborts the application if the specificed amount of memory
  cannot be allocated. Hence, no need to have a NULL check for the
  return value of g_malloc0.

* Removed one if block in VMToolsLogInt since the if check will never
  be TRUE.

* g_strdup returns NULL if the input argument is NULL. Removed
  the unnecessary NULL checks for the input argument to g_strdup at few
  places.

[AppInfo] OVT Coverity fixes.

The return value of Str_Snprintf is not being checked for negative
values and is directly passed to DynBuf_Append which accepts only
positive values. This may cause issues if Str_Snprintf fails. Added
a trivial check.

Initialized appList variable to NULL.

Backout the previous AppInfo OVT Coverity fix.

[AppInfo] OVT Coverity fixes.

The return value of Str_Snprintf is not being checked for negative
values and is directly passed to DynBuf_Append which accepts only
positive values. This may cause issues if Str_Snprintf fails. Added
a trivial check.

Add missing copyright header.

The copyright header is missing in one of the files (Makefile.am)
that is bundled in open-vm-tools tar file. Added the necessary
copyright header.

Common header file change not directly applicable to open-vm-tools.

Common header file change not applicable to open-vm-tools.

Fix Coverity scan issues in open-vm-tools.

Include AppInfo related files in Tools OVT

This changeset contains the necessary changes to include appinfo plugin
related files in Tools OVT bundle.

Fix the deployPkg status files when privateTmp is enabled

Currently, deployPkg status file path is hardcoded to /tmp.  When
privateTmp is enabled in the guest VM, /tmp is hidden from the rest
of the system and is wiped on reboot.  So need use different path to
save the status file.

This change is changing deployPkg status file path from /tmp to /var/log

Backout previous common header file change.

Common header file change not directly applicable to open-vm-tools.

More Coverity annotations for open-vm-tools issues.

Add annotations for additional Coverity-reported issues so that
Coverity scans of ovt will automatically classify these issues
as "intentional."  Such annotations are useful both for internal
use as well as for partners who run Coverity scans of open-vm-tools.

Backout previous default "hgfs.packetSize.large" setting

The default value of config option "hgfs.packetSize.large" is now FALSE.

Common header file change not directly applicable to open-vm-tools.

Due to a quirk the way patches were generated, the 'coomon-agent' directory was not removed as intended, but just contained empty files

Due to a quirk the way patches were generated, the 'coomon-agent' directory was not removed as intended, but just contained empty files