Backout previous change to lib/unicode/unicodeSimpleTypes.c

bora/lib/unicode: fix leak on exit

The hashtable used for encoding name -> IANA table index
lookups was not getting freed.

Also, HashTable_AllocOnce() incorrectly instructs
the hashtable to clear its clientData, which contains
an integer index and not a pointer, using "free".

Common source 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.

Fix a Coverity-reported overrun.

A Coverity scan of open-vm-tools reports a buffer overrun in
Escape_Unescape.  The problem is that Escape_Unescape uses
sizeof('\0') to specify the size of a buffer that consists of
a single character in the variable nulByte (previously named
nullbyte).  However, character literals in C are ints, so
sizeof('\0') is equivalent to sizeof int rather than sizeof char.
Use "sizeof nulByte" instead.

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

Remove the now unused LOGLEVEL_VARIADIC

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

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

LOG variadic: remove LOGLEVEL_VARIADIC [1 of 2]

Mass conversion is complete. It's time to remove the intermediate state.
[1/2]: remove LOGLEVEL_VARIADIC from loglevel_defs.h.  This removes the
backwards-compat path.

Update copyright to reflect previous change.

Fix a trivial memory leak in namespacetool.c.

Fix a memory leak reported by a customer based on their Coverity
scans of 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.

Provide a switch in VMTools configuration to disable guest customization.

This switch can prevent a VI admin from running customization on a VM that
is already customized and give control to guest admin to decide this.

By default the guest customization is enabled.

Provide a switch in VMTools configuration to disable guest customization.

Header file change: adding error enumerations.

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

open-vm-tools: add --enable-valgrind configure option

Add the option "--enable-valgrind" to the ./configure command, and test
if the necessary headers are installed.

Fix crash because xerror in get_monitor_at_window

Sometimes there is an x drawable error while calling get_monitor_at_window.
This error can be safely ignored.

VMware Tools service discovery core plugin changes

First step in creating a new plugin to collect in-guest process related
data over Namespace DB.

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

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

Mapping the deployPkg log error level to g_log warning level.

g_error() is always fatal, resulting in terminating the application.  In
this case the guest customization will fail.  So mapping the deployPkg log
error level to g_log warning level to avoid application termination.

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

Send GUESTRPCPKT_FIELD_FAST_CLOSE when doing one-off RPC via vsock

To work around a vsock/vmci/VMX issue with vsock cleanup, send a special
command that tells the VXM to immediately close a vsock after completing
an RPC.  This minimizes delays/timeouts when sending one-off RPCs rapdily.

Can still run out of vsockets, but they clean up pretty quickly,
so add logic to sleep a bit and retry if ENOBUFS is returned
by connect().

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

Misc. Appinfo plugin cleanup.

- Used 'g' prefix for the global variables.
- Added '%s' at the beginning of each log message.
- Renamed a non-static function with the proper naming convention.
- Fixed documentation for few functions.
- Added '.' at the end of each log message to maintain consistency.
- Used proper '// IN' comments for each function argument.

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

Replace VixTools_ConfigGetBoolean with VMTools_ConfigGetBoolean

Initially came up with a utility function named VixTools_ConfigGetBoolean
to retrieve the boolean value from the config dictionary.  This function
was designed as local/specific to VIX plugin.  Later, VMTools_ConfigGetBoolean
function was designed with the same functionality and is being used everywhere
in the Tools repository.

This changeset gets rid of VixTools_ConfigGetBoolean and replaces
all those occurrences with VMTools_ConfigGetBoolean.

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

Add a new error ASOCKERR_WEBSOCK_UPGRADE_NOT_FOUND in asyncsocket

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

Fix procMgr library for Windows.

- For windows, to retrieve the command line for a target process,
the procMgr library creates a remote thread in the target process
and executes GetCommandLine function.  This approach is OK. But for
few applications that are not built with proper flags, creating a
remote thread may crash the application.  In order to avoid any
issues, it was decided not use the 'remote thread' approach by DEFAULT.

- Refactored the procMgr APIs to take an input argument from the
user if the 'remote thread' approach needs to be used.

- Refactored the procMgr APIs to conditionally use the WMI for
retrieving the command line for a target process.

- New options are provided to the user if the 'remote thread'
and 'WMI' approaches need to be forced.  Modified the VIX tools plugin
to read and honor the new flags from tools.conf

The following are the new flags under guestoperations group.
useRemoteThreadForProcessCommandLine
useWMIForProcessCommandLine

The default value for the above new flags is false. User can set them
to true if needed.

- Re-factored some code in appInfo plugin that calls procMgr library.
The appinfo plugin doesn't really use the commandline.  So, modified the
appinfo plugin to just use the API that doesn't use the complicated
approaches for the listing down the processes.

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

Arm64 backdoor: Disable USE_HYPERCALL for Arm64 tools build.

Undefine USE_HYPERCALL on ESXi-Arm to force the backdoor interface type
to BACKDOOR_INTERFACE_IO.  This is necessary to fix the Arm64 tools build.
This change does not affect x86 (open-vm-tools).

FreeBSD has removed some vnops flags that have never been used.

Two flags that were being tested in the nullfs vnops.c source have
never actually been used by any filesystem layer on top of the nullfs.
FreeBSD is doing a little house cleaning and the VDESC_NOMAP_VPP and
VDESC_VPP_WILLRELE defines are being removed from the source base.

FreeBSD has provided source code changes for the FreeBSD vmblock
driver as they have made in identical nullfs code in the FreeBSD
base code.

Pull request: https://github.com/vmware/open-vm-tools/pull/403

VMTools 11.1.0: update visible copyrights to 2020.

Add hypercall to bora/lib/backdoor.

Use vmcall or vmmcall when they are available.
Removed #include <string.h>, it was added in an earlier version
  but is not needed in the current code.
Limit HostinfoBackdoorGetInterface to x86 architecture.
Fix some indents.

Remove 'VMware Confidential' from license files for publishing

Handle LSI Logic Parallel and BusLogic SCSI on PCI bus 0.

When a LSI Logic Parallel or BusLogic SCSI controller appears on PCI
bus 0, the disk device controller name is not present; the "label" file
is missing.  Adding logic to check if disk device is attached to a
SCSI controller on bus 0 (0000:00:10.0) if a "label" file cannot be
found.  If that is the case, setting the device name to "scsi0.n"
where "n" is the device unit number previously determined.

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

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

Fix broken PowerCLI Invoke-VMScript cmdtlet by an earlier change.
Also fixed a handle leak in VixToolsGetTokenHandleFromTicketID.

Note: Credential type VIX_USER_CREDENTIAL_TICKETED_SESSION, started by
VIX_COMMAND_ACQUIRE_CREDENTIALS, is not built upon VGAuth lib
VGAuth_GenerateSSPIChallenge/VGAuth_ValidateSSPIResponse, impersonation is
done via lib/impersonate/impersonate.c::Impersonate_Do, the loaded user
profile is designed to be unloaded after 5 minutes, but because vix.dll is
linked with stub lib poll whose Poll_CB_RTime() does not do anything, the
profile is never unloaded.

Handle Photon's "/dev/root" mount point when doing disk device mapping

Photon EFI boot VMs have the root disc mounted long before the various
PCI controllers are initialized and the attached devices enumerated.
When looking through the /proc/mounts, the root filesystem mounted
at '/' is displayed as if on device /dev/root/  But there is no "root"
device in /dev, nor in the PCI device tree at /sys/class/block.  Later
Photon builds fabricate a block device at "/dev/root" but again nothing
in the /sys/class/block PCI tree.

For this situation, it is necessary to use the pseudo device's major and
minor number to access the PCI device tree through /sys/dev/block.
The major and minor device number can be extracted from the contents of
/proc/self/mountinfo.

19 0 8:2 / / rw,relatime shared:1 - ext4 /dev/root rw

The 3rd field is the major:minor number and the 9th or 10th, depending
on Linux release, is the pseudo device in this case.

Rectify a log spew in vmsvc logging (vmware-vmsvc-root.log)

When a LSI Logic Parallel SCSI controller sits in PCI bus 0
(SCSI controller 0), the Linux disk device enumeration does not provide
a "label" file with the controller name.  This results in messages like

'GuestInfoGetDiskDevice: Missing disk device name; VMDK mapping
unavailable for "/var/log", fsName: "/dev/sda2"'

repeatedly appearing in the vmsvc logging.  The patch converts what
previously was a warning message to a debug message and thus avoids
the log spew.

This patch has been made available for open-vm-tools 11.0.x on the
open-vm-tools github repo.

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

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

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

FreeBSD kernel API changes with FreeBSD 13.0 (development)

VOP_UNLOCK(): second argument "flags" has been dropped when the kernel
version is >= 1300074.

The timeout(9) interface has been deprecated and must be replaced with
callout(9) (v2) when the kernel version >= 1300067.

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

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

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

lib/file: memory leak in File_WalkDirectoryNext

During a sequence of POSIX readdir calls it is possible for a directory
to be rearranged and a file that was previously reported may be found
again.  The code that protects the caller from seeing this isn't freeing
the file name string if a duplicate is discovered.  Fix this.

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

Back out the previous change to lib/include/str.h

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

More LOG() calls changed to variadic form.

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

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

More LOG() calls changed to variadic form.

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

More LOG() calls changed to variadic form - backdoor.c

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

Merge branch 'stable-11.0.x'

prepare for 11.0.5 release

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

Changes to common header files.
Make ALIGNED() functional for 64-bit Windows

The current implementation of ALIGNED in vm_basic_types.h only
works for compilers that are, or pretend to be, GCC.  However,
it is reasonable to conclude that data specifically aligned
for one OS / compiler should be aligned the same way when built
with other tools or for OSes.

This change updates the ALIGNED macro to correctly function when
building on 64-bit Windows.  If not using GCC-alike, or Windows,
the macro continues to expand to nothing.

The MSVC __declspec construct accepts a limited syntax, basically
requiring an integer literal (see pciPassthruInfo_public.h).  It
must be a prefix operation.

The GCC __attribute__ is more expressive and can be postfix or
prefix, so it is moved to be prefix.

The bulk of the change moves the use of ALIGNED to be prefix in
all places that compile for the Workstation product on Windows.

LOG_ONCE changes to variadic just as LOG did recently.

common header file changes not applicable to open-vm-tools.

Tools: More support for "VMware Tools for Linux-arm64"

Preliminary touch-ups in the main branch to support the upcoming upstreaming
of the "VMware Tools for Linux-arm64" build code.

bora-vmsoft/vgauth/service/servicePosix.c:
  The glibc on arm64 does not have <sys/io.h>.  It makes sense because I/O
  ports are an x86-ism (for more details, sees
   https://bugzilla.redhat.com/show_bug.cgi?id=1116162#c1 ).  Instead of
  making the inclusion conditions more complex, It was noticed that none
  of the symbols defined in <sys/io.h> and <sys/sysinfo.h> are used in
  this C file.  So these inclusions are not necessary.

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

Remove overly complex ifdefs in the dndcp plugin.

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.

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

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

Implement better logging for deploypkg plugin if vmware-imc cannot be created.

If vmware-imc cannot be created, log the error message and also redirect the
customization log to tools log.

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

A previous change incorrectly deals with the capacity of file system - the
file system size - and the amount of free space the file system has.

Revert the change; fix a function header; comments to make it clear
what not to break in the future.

Clean up and make consistent [trivial]

Make the ASSERTs consitent in their coding; clean up a few things.

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.

File locking depriv, part 2

Remove the device locking code.

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

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

Changes to common source files not directly applicable to open-vm-tools.
Depriv the file locking code, part 1

This is the first change in a series of changes toward removing the
need to be root to use the file locking code.  Here, doing some cleanup
and separate the file locking code from the device locking code.

Another later change will depriv the file locking code.

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

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

[Dynbuf] Rename DynBuf_GrowToFit to DynBuf_EnsureMinSize

As suggested, GrowToFit isn't a verb, suggest renaming the function to
DynBuf_EnsureMinSize, and change the description as "Ensure that the
size of the DynBuf is at least 'size'"

Change all references of DynBuf_GrowToFit to DynBuf_EnsureMinSize.

Remove sslDirect.c from open-vm-tools.

It is no longer needed.

[Dynbuf] Fix minor bug in DynBuf_GrowToFit

DynBuf_GrowToFit tries to provide a dynbuf with size greater than the
given size.  However, the second argument passed to DynBuf_Enlarge doesn't
match the intention.  The size should be directly passed, not the
difference between the target and current size.

Trivial - Add Copyright symbol '(c)' to tools.conf.

Missed this in the previous review.
Legal guidelines are to use '(c)' in ASCII files.

Add Copyright notice to bora-vmsoft/install/common/tools.conf.

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