Fix ordering of HGFS opcode for search read V4 backwards compatibiliy

Since change notification is released as of Beta 1 the previous change
of inserting the search read opcode ahead of this will lead to
incompatibily for Beta 2 servers against Beta 1 tools.

Make the continuity seamless by keeping the change notification
opcode ahead of the search read V4. This means the HGFS server dispatch
handler must check for a NULL entry in its handler dispatch table now.
(Or we could put a dummy dispatch handler which always returns protocol
error, but I think NULL is okay.)
The change notification opcode is only for requests sent by the server
to the guest clients in response to a directory change event.

Signed-off-by: Marcelo Vanzin <mvanzin@vmware.com>

diff --git a/open-vm-tools/lib/hgfsServer/hgfsServerParameters.c b/open-vm-tools/lib/hgfsServer/hgfsServerParameters.c
index ea133e799fc3733837a05e401527c5008bfdb9b2..2905985dbc87c4b1aed06a20593731eaa77fb7a3 100644 (file)
--- a/open-vm-tools/lib/hgfsServer/hgfsServerParameters.c
+++ b/open-vm-tools/lib/hgfsServer/hgfsServerParameters.c
@@ -104,8 +104,8 @@ static HgfsCapability hgfsDefaultCapabilityTable[] =
    {HGFS_OP_WRITE_FAST_V4,         HGFS_REQUEST_NOT_SUPPORTED},
    {HGFS_OP_SET_WATCH_V4,          HGFS_REQUEST_NOT_SUPPORTED},
    {HGFS_OP_REMOVE_WATCH_V4,       HGFS_REQUEST_NOT_SUPPORTED},
-   {HGFS_OP_SEARCH_READ_V4,        HGFS_REQUEST_NOT_SUPPORTED},
    {HGFS_OP_NOTIFY_V4,             HGFS_REQUEST_NOT_SUPPORTED},
+   {HGFS_OP_SEARCH_READ_V4,        HGFS_REQUEST_NOT_SUPPORTED},
    {HGFS_OP_OPEN_V4,               HGFS_REQUEST_NOT_SUPPORTED},
    {HGFS_OP_ENUMERATE_STREAMS_V4,  HGFS_REQUEST_NOT_SUPPORTED},
    {HGFS_OP_GETATTR_V4,            HGFS_REQUEST_NOT_SUPPORTED},

diff --git a/open-vm-tools/lib/include/hgfsProto.h b/open-vm-tools/lib/include/hgfsProto.h
index c2d93973013696470ed8d37a01be377267af814e..3dd66688882654e9a059f7f524790743eb4fedaa 100644 (file)
--- a/open-vm-tools/lib/include/hgfsProto.h
+++ b/open-vm-tools/lib/include/hgfsProto.h
@@ -128,8 +128,8 @@ typedef enum {
    HGFS_OP_WRITE_FAST_V4,         /* Write */
    HGFS_OP_SET_WATCH_V4,          /* Start monitoring directory changes. */
    HGFS_OP_REMOVE_WATCH_V4,       /* Stop monitoring directory changes. */
-   HGFS_OP_SEARCH_READ_V4,        /* Read V4 directory entries. */
    HGFS_OP_NOTIFY_V4,             /* Notification for a directory change event. */
+   HGFS_OP_SEARCH_READ_V4,        /* Read V4 directory entries. */
    HGFS_OP_OPEN_V4,               /* Open file */
    HGFS_OP_ENUMERATE_STREAMS_V4,  /* Enumerate alternative named streams for a file. */
    HGFS_OP_GETATTR_V4,            /* Get file attributes */