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

Add WarningToHost() and WarningToGuest() functions

This change is needed to address the privacy and security changes that
are required so that vmware library warnings can be forwarded to the
host side selectively.

For instance, if a warning message is sent to VMX, the user name must be
stripped from the message.

Fix CONSTANT_EXPRESSION_RESULT in TimeUtil_StringToDate().

Red Hat ran coverity scan on open-vm-tools and found an occurrence
of CONSTANT_EXPRESSION_RESULT in TimeUtil_StringToDate().

[10.3.10] Update copyright year to 2019.

Update the copyright year to 2019 for ALL open-vm-tools destined files
modified since the release of 10.3.5.

Bail out of vmtoolsd early when there are RPC errors.

VMX state machine could give up quiescing operation for various
reasons when vmtoolsd is busy performing necessary state transitions.
Once VMX gives up quiescing operation, there is no point in
vmtoolsd continuing with it. vmtoolsd should also give up the
operation asap. The vmtoolsd process can detect VMX state machine
change when it gets errors sending VMBACKUP_PROTOCOL_EVENT_SET RPC.

RPC errors are only used as a trigger to abort the operation.
We ignore the RPC errors that might occur after aborting the
operation.

Don't send a backup manifest when aborting a Linux quiesced snapshot.

When taking a Linux quiesced snapshot, communication failures between
VMX and VMTools may result in VMTools sending a genericManifest event
message after the quiesced snapshot operation has been aborted.  If
this happens, VMX will send an error back to VMTools, which in turn
causes VMTools not to send genericManifest messages on subsequent
quiesced snapshots even if the host supports such messages.

One aspect of the implementation that gives rise to this behavior is
the use of the sync provider's snapshotDone function to undo a
quiescing operation.  Specifically, if VMTools aborts a quiesced
snapshot when the file system is quiesced, the quiescing must be
undone.  Currently, this is handled by calling the sync provider's
snapshotDone function.  This is the same function that is called to
complete the quiescing snapshot protocol when it is successful.  In
some respects this makes sense, since in either case snapshotDone
unquiesces the file system.  However, architecturally and conceptually,
it seems useful to distinguish between the action to be taken in the
successful case versus the aborting case.  It's also useful to do so
in practice, because the successful case sends the genericManifest
event to notify the host there is a backup manifest file, while the
aborting case should not do that.

To address the issue, add an "undo" function for the Linux sync
provider.  The undo function is called instead of snapshotDone as
part of aborting a quiesced snapshot in which the file system is
quiesced at the time of the abort.

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

Common source file change not applilcable to open-vm-tools.

Common source file change 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 header file change not applicable to open-vm-tools

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

Record VMware Tools version 10.3.7 used for a patch.

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

Modernize diskInfo RPC

Rewrite diskInfo RPC to allow it to be backwards and forwards compatible.
step 1 -- re-org existing code.

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

Special-case profile loading for StartProgram

Keep the user's profile loaded for the life of the
program, to avoid races where its unloaded too quickly.

Switch some stringxx headers to use angle brackets instead of double quotes

Code clean-up - bora-vmsoft/lib/procMgr/procMgrPosix.c

Recent discussions about setting "errno" to ENOENT when either
getpwuid_r() or getpwnam_r() return 0 (success) if there is no passwd
entry for the user have vacillated.

Since none of the current callers of the routines in procMgrPosix.c
query the contents of "errno", the current consensus is to simply log
a warning message along with reason for the failure.

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

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

Changes to common header files not directly appilcable to open-vm-tools.

Style: Fix parameter alignment.

Changes to common header files not applicable to open-vm-tools.

Updates to the DeployPkg guest customization source code.

Changes to common header files; not applicable to open-vm-tools.

Changes to common header files not applicable to open-vm-tools.

Header file guestcust-events.h moved to a new location.

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

General code clean-up.

bora/lib/misc/posixPosix.c:
Treat local variables "len" consistently as "size_t" type in
Posix_Getmntent_r()

bora-vmsoft/apps/toolbox-cmd/toolboxcmd-shrink.c:
Improve readability of error handling logic in ShrinkDoWipeAndShrink()
and remove another line of dead code.

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

Record VMware Tools version 10.3.10 as TOOLS_VERSION_JACKHAMMER_UPDATE2

Back out previous change to plugins/vix/vixTools.c.

Too many subtle issues to trust this one without more testing.

Cleanup -Wno-incompatible-pointer-types-discards-qualifiers

Details

Both gcc and clang (on Mac) warn for pointer-incompatible
assignments.  It appears that gcc's -Wwarn-strings isn't working
properly, because it produces no warnings for the following
assignment:

char *potentialSegV = ".rodata-based string";

However, clang does.

This change addresses three problems in bora/lib that produce build
failures when -Wincompatible-pointer-types-discards-qualifiers is
enabled when using clang (one wonders how many other such potential
problems exist?)

o bsd_vsnprintf.c

The 'cp' pointer is assigned the address of a constant string.
Since there are writes through 'cp', the quickest solution is to
declare a local, non-const, array variable that holds the string
"(null)".

o unicodeSimpleTypes.c

This change, the easiest of the three, marks the 'char *' field as
'const'.

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

Backout previous change to vm_basic_asm.h; not 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.

Revert previous change to vmci_sockets.h.

Fix NULL pointer dereference and remove three lines of dead code.

QbitLogic has run [Code]Ai on the open-vm-tools source code and
and notified VMware of a potential NULL pointer reference and four
pieces of dead code.

See https://github.com/vmware/open-vm-tools/pull/247

bora-vmsoft/services/vmtoolsd/pluginMgr.c
The proposed fix conditionally emitted a log message.  That fix has
been reworked to consistently emit a log message with conditional
content.

bora/lib/misc/posixPosix.c
bora-vmsoft/apps/toolbox-cmd/toolboxcmd-shrink.c
Remove one line of dead code from each source file.

bora-vmsoft/lib/procMgr/procMgrPosix.c:
Suggested dead code fix actually pointed to a bug where errno
was not getting updated as intended.

bora-vmsoft/services/plugins/guestInfo/perfMonLinux.c:
Suggested dead code is no longer dead; change not applicable.

Remove erroneous defines of __FreeBSD__ & __VMKTCPIP__ from vmci_sockets.h

Add VMware Tools version 10.3.6 to vm_tools_version.h

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

special case profile loading for StartProgram

For StartProgram, we need to be sure there's no race in the profile
unloading.  So keep it open through the life of the started program.

See related vgauth change.

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

Fix the issue that Deploypkg log files of linux should not be world readable.

lib/file: Memory leak in File_ListDirectory when directory is empty

If the directory is empty, calloc of zero still succeeds and a memory
leak results.

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

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

lib/file: Improve File_ListDirectory and File_WalkDirectory*

The File_WalkDirectory implementation has an initial latency (before
file names are available) that is unnecessary.  Using the unicode
library, when not necessary, adds a huge amount of memory usage and
wastes CPU time.

These routines duplicate quite a bit of code.

Rewrite File_ListDirectory to use File_WalkDirectory*.

Rewrite File_WalkDirectory*:
 - Return files as they are discovered, not after the entire directory
   contents are parsed. If one decides to stop early, they don't have
   to pay the price for the entire directory contents.
 - Preserve the protection from duplicate file names.  While Windows
   locks a directory during a content tranverse, POSIXen does nott. Code
   is here in the lib/file for quite some time so that the numerous
   callers do not have to handle dealing with dups.
 - Avoid using the unicode library on platforms where we can, this avoid
   most of the eggregious memory usage.
 - Don't use Dynbuf... Handling things directly is more efficient.

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

Bump tools version to 11.0.0.

The next VMWare Tools product release is to be version 11.0.0.
Switching the version of tools from the current development version 10.4.0
to 11.0.0.

Remove GLib 2.32 deprecated APIs from tools

Replace the GThread, GCond and GMutex APIs deprecated in GLib version
2.32 in the VMware Tools source.

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

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

Disable sending toolsd debug log level messages to host

After the security and privacy reviews of the toolsd logging,
the debug level messages to host have been turned off.

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

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

Remove grabbitmqproxy and certtool from tar installer, OSPs and open-vm-tools.

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

Add a new GOS definition for darwin19_64 (macOS 10.15) in HWV >= 17

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

Adding the libappmonitor source to the open-vm-tools distribution.

In response to customer requests that the libappmonitor library from
the VMwares Guest SDK be opensourced, the libappmonitor code has been
bundled in open-vm-tools.

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

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

Provide a distroName, when available, when the lsb_release cmd fails.

The default, in Linux, is "Linux". If a distoName was present in the
lsb release description file, it should be used.

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

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

Remove Common Agent Framework (CAF) from Tools - Linux Changes

The CAF files have been removed from VMWare Tools and the open-vm-tools
source.

Enhance the vm-support-lx script to capture guest customization related logs
for the guest.

Security concern: Certificates contain identifying information

Do not log the contents of certificates which contain identifying
information such as e-mail addresses.

Merge branch 'stable-10.3.x'

release notes for 10.3.5

add 10.3.5 changes to ChangeLog

bump version and build number

sync deploypkg files

Add "#include <string.h>" to services/vmtoolsd/toolsHangDetector.c.

Common header file change.

Remove macro definitions of MAX_OS_NAME_LEN and MAX_OS_FULLNAME_LEN
from lib/misc/hostinfoInt.h

Hide static function declaration for Linux from FreeBSD and Solaris OVT

A recent change introduced a new static function
HgfsInvalidateParentsChildren() in bora-vmsoft/hgfs/fuse/cache.c. That
function is defined and called only in Linux builds.

The static function declaration should not be visible on FreeBSD OVT
builds.

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

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

GuestMapper: Detailed data fixes

The guestInfo detailed data for Photon was being reported incorrectly,
sometimes adding trailing whitespace when not needed.

The problem was how the release file was processed. It was being
processed (open/read/code/close) multiple times, and wasn't separating
each of the fields as it should. Fixed this.

Adding logging of what is sent by the guest; how things are mapped.

Update the tools side code to send VMX the health status event.

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

VIGOR and RPCI definitions for the tools hang detector events.

Define an array of ToolsHealthEvent to record the last N tools hang
events. By keeping a list of historical events since the power on,
we help ourselves trouble shooting guest/toolsd issues.

Use the first array element for the latest event. This simplifies
implementation as the DynArray can be easily capped by setting the
array count. This requires us to add a PushFront function to add
new element to the front of the DynArray.

Added the RPCI handler for the tools hang detector RPCI messages.

Added feature switch Tools_Update_Health.

Common header file change not 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 header file change not applicable to open-vm-tools.

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

Add more logs for clipboard delay rendering

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

Backout immediately previous change; not applicable to open-vm-tools.

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

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

Replace function File_Rename() call with File_Move()in dndCommon.c

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

Re-apply the DnD changes previously backed out.

On Windows, the CDR tmp files under folder "VMwareDnD" will be kept
permanently in some cases after drag & drop.i

Fork sha1/md5 for kernel/user usage

Kernel specific sha1/md5 routines have been removed from lib/misc/sha1.c.

Backout the previous changes to DnD temp file clean up..

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

On Windows, the CDR tmp files under folder "VMwareDnD" will be kept
permanently in some cases after drag & drop.

Root cause: The existing strategy is to delete the folder before next reboot
of the machine, which is implemented through writing Windows HKLM registry.
However, for the non-administrator user, that user has no permission to
write the registry which in turn results in the temp files not being removed.

Solution: The temp files will be removed when the user disconnect the remote
desktop/app. The details are:
 1. Client will remove the temp folder when remote desktop/app is disconnected
    (rmks exits). Server will remove the temp folder when mksvchServer plugin
    gets "Not Ready" notification which means mksvchanServer is disconnected
    from the  mksvchanClient.
 2. Use prefix "Horizon_xxxx(pid)-" to distinguish if the temp folder is
    being used by DnD or not. For Client, the pid is the rmks process id,
    for Server, the pid is Clipboard pid.