Revert "case insensitive ".dll" for VST plugins - fixes #6285"
[ardour.git] / libs / ardour / vst_info_file.cc
index 50d94850d6d346a94b3ed5e58b781b171607fd1d..2d68421103d0d14930f8ff9d5925ba333d39540f 100644 (file)
@@ -34,7 +34,6 @@
 #include <stddef.h>
 #include <stdio.h>
 #include <string.h>
-#include <libgen.h>
 
 #include <glib.h>
 #include <glib/gstdio.h>
 
 #include "pbd/error.h"
 
+#ifndef VST_SCANNER_APP
+#include "ardour/plugin_manager.h" // scanner_bin_path
+#include "ardour/rc_configuration.h"
+#include "ardour/system_exec.h"
+#endif
+
+#include "ardour/filesystem_paths.h"
 #include "ardour/linux_vst_support.h"
+#include "ardour/plugin_types.h"
 #include "ardour/vst_info_file.h"
 
 #define MAX_STRING_LEN 256
+#define PLUGIN_SCAN_TIMEOUT (Config->get_vst_scan_timeout()) // in deciseconds
+
+
+/* CACHE FILE PATHS */
+#define EXT_BLACKLIST ".fsb"
+#define EXT_ERRORFILE ".err"
+#define EXT_INFOFILE  ".fsi"
+
+#ifdef PLATFORM_WINDOWS
+#define PFX_DOTFILE   ""
+#else
+#define PFX_DOTFILE   "."
+#endif
+
 
 using namespace std;
+#ifndef VST_SCANNER_APP
+namespace ARDOUR {
+#endif
+
+/* prototypes */
+#ifdef WINDOWS_VST_SUPPORT
+#include <fst.h>
+static bool
+vstfx_instantiate_and_get_info_fst (const char* dllpath, vector<VSTInfo*> *infos, int uniqueID);
+#endif
+
+#ifdef LXVST_SUPPORT
+static bool vstfx_instantiate_and_get_info_lx (const char* dllpath, vector<VSTInfo*> *infos, int uniqueID);
+#endif
+
+/* ID for shell plugins */
+static int vstfx_current_loading_id = 0;
+
+
+
+/* *** CACHE FILE PATHS *** */
+
+static string
+vstfx_cache_file (const char* dllpath, int personal, const char *ext)
+{
+       string dir;
+       if (personal) {
+               dir = get_personal_vst_info_cache_dir();
+               // TODO prefix path relative to scan-root to avoid duplicates
+       } else {
+               dir = Glib::path_get_dirname (std::string(dllpath));
+       }
+
+       stringstream s;
+       s << PFX_DOTFILE << Glib::path_get_basename (dllpath) << ext;
+       return Glib::build_filename (dir, s.str ());
+}
+
+static string
+vstfx_blacklist_path (const char* dllpath, int personal)
+{
+       string dir;
+       if (personal) {
+               dir = get_personal_vst_blacklist_dir();
+       } else {
+               dir = Glib::path_get_dirname (std::string(dllpath));
+       }
 
+       stringstream s;
+       s << PFX_DOTFILE << Glib::path_get_basename (dllpath) << EXT_BLACKLIST;
+       return Glib::build_filename (dir, s.str ());
+}
+
+static string
+vstfx_infofile_path (const char* dllpath, int personal)
+{
+       return vstfx_cache_file(dllpath, personal, EXT_INFOFILE);
+}
+
+#ifndef VST_SCANNER_APP
+static string
+vstfx_errorfile_path (const char* dllpath, int personal)
+{
+       return vstfx_cache_file(dllpath, personal, EXT_ERRORFILE);
+}
+#endif
+
+
+/* *** MEMORY MANAGEMENT *** */
+
+/** cleanup single allocated VSTInfo */
+static void
+vstfx_free_info (VSTInfo *info)
+{
+       for (int i = 0; i < info->numParams; i++) {
+               free (info->ParamNames[i]);
+               free (info->ParamLabels[i]);
+       }
+
+       free (info->name);
+       free (info->creator);
+       free (info->Category);
+       free (info->ParamNames);
+       free (info->ParamLabels);
+       free (info);
+}
+
+/** reset vector */
+static void
+vstfx_clear_info_list (vector<VSTInfo *> *infos)
+{
+       for (vector<VSTInfo *>::iterator i = infos->begin(); i != infos->end(); ++i) {
+               vstfx_free_info(*i);
+       }
+       infos->clear();
+}
+
+
+
+/* *** CACHE FILE I/O *** */
+
+/** Helper function to read a line from the cache file
+ * @return newly allocated string of NULL
+ */
 static char *
 read_string (FILE *fp)
 {
@@ -84,15 +208,7 @@ read_int (FILE* fp, int* n)
        return (sscanf (p, "%d", n) != 1);
 }
 
-static void
-vstfx_clear_info_list (vector<VSTInfo *> *infos)
-{
-       for (vector<VSTInfo *>::iterator i = infos->begin(); i != infos->end(); ++i) {
-               vstfx_free_info(*i);
-       }
-       infos->clear();
-}
-
+/** parse a plugin-block from the cache info file */
 static bool
 vstfx_load_info_block(FILE* fp, VSTInfo *info)
 {
@@ -107,6 +223,17 @@ vstfx_load_info_block(FILE* fp, VSTInfo *info)
        if (read_int (fp, &info->hasEditor)) return false;
        if (read_int (fp, &info->canProcessReplacing)) return false;
 
+       /* backwards compatibility with old .fsi files */
+       if (info->wantMidi == -1) {
+               info->wantMidi = 1;
+       }
+
+       if ((info->numParams) == 0) {
+               info->ParamNames = NULL;
+               info->ParamLabels = NULL;
+               return true;
+       }
+
        if ((info->ParamNames = (char **) malloc(sizeof(char*)*info->numParams)) == 0) {
                return false;
        }
@@ -127,6 +254,7 @@ vstfx_load_info_block(FILE* fp, VSTInfo *info)
        return true;
 }
 
+/** parse all blocks in a cache info file */
 static bool
 vstfx_load_info_file (FILE* fp, vector<VSTInfo*> *infos)
 {
@@ -140,7 +268,7 @@ vstfx_load_info_file (FILE* fp, vector<VSTInfo*> *infos)
                } else {
                        int plugin_cnt = 0;
                        vstfx_free_info(info);
-                       if (read_int (fp, &plugin_cnt)) {
+                       if (!read_int (fp, &plugin_cnt)) {
                                for (int i = 0; i < plugin_cnt; i++) {
                                        if ((info = (VSTInfo*) calloc (1, sizeof (VSTInfo))) == 0) {
                                                vstfx_clear_info_list(infos);
@@ -203,7 +331,7 @@ vstfx_write_info_file (FILE* fp, vector<VSTInfo *> *infos)
                 * plugins contained in this shell
                 */
                vstfx_write_info_block(fp, *x);
-               fprintf( fp, "%d\n", infos->size() - 1 );
+               fprintf( fp, "%d\n", (int)infos->size() - 1 );
                ++x;
                /* Now write out the info for each plugin */
                for (; x != infos->end(); ++x) {
@@ -212,35 +340,100 @@ vstfx_write_info_file (FILE* fp, vector<VSTInfo *> *infos)
        } else if (infos->size() == 1) {
                vstfx_write_info_block(fp, infos->front());
        } else {
-               PBD::warning << "Zero plugins in VST." << endmsg; // XXX here?
+               PBD::error << "Zero plugins in VST." << endmsg; // XXX here? rather make this impossible before if it ain't already.
        }
 }
 
-static string
-vstfx_infofile_path (char* dllpath, int personal)
+
+/* *** CACHE AND BLACKLIST MANAGEMENT *** */
+
+/* return true if plugin is blacklisted or has an invalid file extension */
+static bool
+vstfx_blacklist_stat (const char *dllpath, int personal)
 {
-       string dir;
-       if (personal) {
-               dir = Glib::build_filename (Glib::get_home_dir (), ".fst");
+       if (strstr (dllpath, ".so" ) == 0 && strstr(dllpath, ".dll") == 0) {
+               return true;
+       }
+       string const path = vstfx_blacklist_path (dllpath, personal);
+
+       if (Glib::file_test (path, Glib::FileTest (Glib::FILE_TEST_EXISTS | Glib::FILE_TEST_IS_REGULAR))) {
+               struct stat dllstat;
+               struct stat fsbstat;
 
-               /* If the directory doesn't exist, try to create it */
-               if (!Glib::file_test (dir, Glib::FILE_TEST_IS_DIR)) {
-                       if (g_mkdir (dir.c_str (), 0700)) {
-                               return 0;
+               if (stat (dllpath, &dllstat) == 0 && stat (path.c_str(), &fsbstat) == 0) {
+                       if (dllstat.st_mtime > fsbstat.st_mtime) {
+                               /* plugin is newer than blacklist file */
+                               return true;
                        }
                }
+               /* stat failed or plugin is older than blacklist file */
+               return true;
+       }
+       /* blacklist file does not exist */
+       return false;
+}
 
-       } else {
-               dir = Glib::path_get_dirname (std::string(dllpath));
+/* return true if plugin is blacklisted, checks both personal
+ * and global folder */
+static bool
+vstfx_check_blacklist (const char *dllpath)
+{
+       if (vstfx_blacklist_stat(dllpath, 0)) return true;
+       if (vstfx_blacklist_stat(dllpath, 1)) return true;
+       return false;
+}
+
+/* create blacklist file, preferably in same folder as the
+ * plugin, fall back to personal folder in $HOME
+ */
+static FILE *
+vstfx_blacklist_file (const char *dllpath)
+{
+       FILE *f;
+       if ((f = fopen (vstfx_blacklist_path (dllpath, 0).c_str(), "wb"))) {
+#ifndef NDEBUG
+       PBD::info << "Blacklisted VST: '" << vstfx_blacklist_path (dllpath, 0) << "'" << endmsg;
+#endif
+               return f;
        }
+#ifndef NDEBUG
+       PBD::info << "Blacklisted VST: '" << vstfx_blacklist_path (dllpath, 1) << "'" << endmsg;
+#endif
+       return fopen (vstfx_blacklist_path (dllpath, 1).c_str(), "wb");
+}
 
-       stringstream s;
-       s << "." << Glib::path_get_basename (dllpath) << ".fsi";
-       return Glib::build_filename (dir, s.str ());
+/** mark plugin as blacklisted */
+static bool
+vstfx_blacklist (const char *dllpath)
+{
+       FILE *f = vstfx_blacklist_file(dllpath);
+       if (f) {
+               fclose(f);
+               return true;
+       }
+       return false;
+}
+
+/** mark plugin as not blacklisted */
+static void
+vstfx_un_blacklist (const char *dllpath)
+{
+       ::g_unlink(vstfx_blacklist_path (dllpath, 0).c_str());
+       ::g_unlink(vstfx_blacklist_path (dllpath, 1).c_str());
+}
+
+/** remove info file from cache */
+static void
+vstfx_remove_infofile (const char *dllpath)
+{
+       ::g_unlink(vstfx_infofile_path (dllpath, 0).c_str());
+       ::g_unlink(vstfx_infofile_path (dllpath, 1).c_str());
 }
 
+/** helper function, check if cache is newer than plugin
+ * @return path to cache file */
 static char *
-vstfx_infofile_stat (char *dllpath, struct stat* statbuf, int personal)
+vstfx_infofile_stat (const char *dllpath, struct stat* statbuf, int personal)
 {
        if (strstr (dllpath, ".so" ) == 0 && strstr(dllpath, ".dll") == 0) {
                return 0;
@@ -250,11 +443,6 @@ vstfx_infofile_stat (char *dllpath, struct stat* statbuf, int personal)
 
        if (Glib::file_test (path, Glib::FileTest (Glib::FILE_TEST_EXISTS | Glib::FILE_TEST_IS_REGULAR))) {
 
-               /* info file exists in same location as the shared object, so
-                  check if its current and up to date
-               */
-
-
                struct stat dllstat;
 
                if (stat (dllpath, &dllstat) == 0) {
@@ -270,9 +458,10 @@ vstfx_infofile_stat (char *dllpath, struct stat* statbuf, int personal)
        return 0;
 }
 
-
+/** cache file for given plugin
+ * @return FILE of the .fsi cache if found and up-to-date*/
 static FILE *
-vstfx_infofile_for_read (char* dllpath)
+vstfx_infofile_for_read (const char* dllpath)
 {
        struct stat own_statbuf;
        struct stat sys_statbuf;
@@ -299,19 +488,28 @@ vstfx_infofile_for_read (char* dllpath)
        return rv;
 }
 
+/** helper function for \ref vstfx_infofile_for_write
+ * abstract global and personal cache folders
+ */
 static FILE *
-vstfx_infofile_create (char* dllpath, int personal)
+vstfx_infofile_create (const char* dllpath, int personal)
 {
        if (strstr (dllpath, ".so" ) == 0 && strstr(dllpath, ".dll") == 0) {
                return 0;
        }
 
        string const path = vstfx_infofile_path (dllpath, personal);
-       return fopen (path.c_str(), "w");
+#ifndef NDEBUG
+       PBD::info << "Creating VST cache file " << path << endmsg;
+#endif
+       return fopen (path.c_str(), "wb");
 }
 
+/** newly created cache file for given plugin
+ * @return FILE for the .fsi cache, NULL if neither personal,
+ * nor global cache folder is writable */
 static FILE *
-vstfx_infofile_for_write (char* dllpath)
+vstfx_infofile_for_write (const char* dllpath)
 {
        FILE* f;
 
@@ -322,8 +520,49 @@ vstfx_infofile_for_write (char* dllpath)
        return f;
 }
 
+/** check if cache-file exists, is up-to-date and parse cache file
+ * @param infos [return] loaded plugin info
+ * @return true if .fsi cache was read successfully, false otherwise
+ */
+static bool
+vstfx_get_info_from_file(const char* dllpath, vector<VSTInfo*> *infos)
+{
+       FILE* infofile;
+       bool rv = false;
+       if ((infofile = vstfx_infofile_for_read (dllpath)) != 0) {
+               rv = vstfx_load_info_file(infofile, infos);
+               fclose (infofile);
+               if (!rv) {
+                       PBD::warning << "Cannot get VST information form " << dllpath << ": info file load failed." << endmsg;
+               }
+       }
+       return rv;
+}
+
+
+
+/* *** VST system-under-test methods *** */
+
+static
+bool vstfx_midi_input (VSTState* vstfx)
+{
+       AEffect* plugin = vstfx->plugin;
+
+       int const vst_version = plugin->dispatcher (plugin, effGetVstVersion, 0, 0, 0, 0.0f);
+
+       if (vst_version >= 2) {
+               /* should we send it VST events (i.e. MIDI) */
+
+               if ((plugin->flags & effFlagsIsSynth) || (plugin->dispatcher (plugin, effCanDo, 0, 0, const_cast<char*> ("receiveVstEvents"), 0.0f) > 0)) {
+                       return true;
+               }
+       }
+
+       return false;
+}
+
 static
-int vstfx_can_midi (VSTState* vstfx)
+bool vstfx_midi_output (VSTState* vstfx)
 {
        AEffect* plugin = vstfx->plugin;
 
@@ -332,16 +571,58 @@ int vstfx_can_midi (VSTState* vstfx)
        if (vst_version >= 2) {
                /* should we send it VST events (i.e. MIDI) */
 
-               if ((plugin->flags & effFlagsIsSynth) || (plugin->dispatcher (plugin, effCanDo, 0, 0,(void*) "receiveVstEvents", 0.0f) > 0)) {
-                       return -1;
+               if (   (plugin->dispatcher (plugin, effCanDo, 0, 0, const_cast<char*> ("sendVstEvents"), 0.0f) > 0)
+                      || (plugin->dispatcher (plugin, effCanDo, 0, 0, const_cast<char*> ("sendVstMidiEvent"), 0.0f) > 0)
+                        ) {
+                       return true;
                }
        }
 
        return false;
 }
 
-static VSTInfo *
-vstfx_info_from_plugin (VSTState* vstfx)
+/** simple 'dummy' audiomaster callback to instantiate the plugin
+ * and query information
+ */
+static intptr_t
+simple_master_callback (AEffect *, int32_t opcode, int32_t, intptr_t, void *ptr, float)
+{
+       const char* vstfx_can_do_strings[] = {
+               "supplyIdle",
+               "sendVstTimeInfo",
+               "sendVstEvents",
+               "sendVstMidiEvent",
+               "receiveVstEvents",
+               "receiveVstMidiEvent",
+               "supportShell",
+               "shellCategory",
+               "shellCategorycurID"
+       };
+       const int vstfx_can_do_string_count = 9;
+
+       if (opcode == audioMasterVersion) {
+               return 2400;
+       }
+       else if (opcode == audioMasterCanDo) {
+               for (int i = 0; i < vstfx_can_do_string_count; i++) {
+                       if (! strcmp(vstfx_can_do_strings[i], (const char*)ptr)) {
+                               return 1;
+                       }
+               }
+               return 0;
+       }
+       else if (opcode == audioMasterCurrentId) {
+               return vstfx_current_loading_id;
+       }
+       else {
+               return 0;
+       }
+}
+
+
+/** main plugin query and test function */
+static VSTInfo*
+vstfx_parse_vst_state (VSTState* vstfx)
 {
        assert (vstfx);
 
@@ -354,14 +635,26 @@ vstfx_info_from_plugin (VSTState* vstfx)
          fail to implement getVendorString, and so won't stuff the
          string with any name*/
 
-       char creator[65] = "Unknown\0";
+       char creator[65] = "Unknown";
+       char name[65] = "";
 
        AEffect* plugin = vstfx->plugin;
 
-       info->name = strdup (vstfx->handle->name);
+
+       plugin->dispatcher (plugin, effGetEffectName, 0, 0, name, 0);
+
+       if (strlen(name) == 0) {
+               plugin->dispatcher (plugin, effGetProductString, 0, 0, name, 0);
+       }
+
+       if (strlen(name) == 0) {
+               info->name = strdup (vstfx->handle->name);
+       } else {
+               info->name = strdup (name);
+       }
 
        /*If the plugin doesn't bother to implement GetVendorString we will
-         have pre-stuffed the string with 'Unkown' */
+         have pre-stuffed the string with 'Unknown' */
 
        plugin->dispatcher (plugin, effGetVendorString, 0, 0, creator, 0);
 
@@ -374,13 +667,29 @@ vstfx_info_from_plugin (VSTState* vstfx)
                info->creator = strdup (creator);
        }
 
+
+       switch (plugin->dispatcher (plugin, effGetPlugCategory, 0, 0, 0, 0))
+       {
+               case kPlugCategEffect:         info->Category = strdup ("Effect"); break;
+               case kPlugCategSynth:          info->Category = strdup ("Synth"); break;
+               case kPlugCategAnalysis:       info->Category = strdup ("Anaylsis"); break;
+               case kPlugCategMastering:      info->Category = strdup ("Mastering"); break;
+               case kPlugCategSpacializer:    info->Category = strdup ("Spacializer"); break;
+               case kPlugCategRoomFx:         info->Category = strdup ("RoomFx"); break;
+               case kPlugSurroundFx:          info->Category = strdup ("SurroundFx"); break;
+               case kPlugCategRestoration:    info->Category = strdup ("Restoration"); break;
+               case kPlugCategOfflineProcess: info->Category = strdup ("Offline"); break;
+               case kPlugCategShell:          info->Category = strdup ("Shell"); break;
+               case kPlugCategGenerator:      info->Category = strdup ("Generator"); break;
+               default:                       info->Category = strdup ("Unknown"); break;
+       }
+
        info->UniqueID = plugin->uniqueID;
 
-       info->Category = strdup("None"); /* XXX */
        info->numInputs = plugin->numInputs;
        info->numOutputs = plugin->numOutputs;
        info->numParams = plugin->numParams;
-       info->wantMidi = vstfx_can_midi(vstfx);
+       info->wantMidi = (vstfx_midi_input(vstfx) ? 1 : 0) | (vstfx_midi_output(vstfx) ? 2 : 0);
        info->hasEditor = plugin->flags & effFlagsHasEditor ? true : false;
        info->canProcessReplacing = plugin->flags & effFlagsCanReplacing ? true : false;
        info->ParamNames = (char **) malloc(sizeof(char*)*info->numParams);
@@ -405,184 +714,369 @@ vstfx_info_from_plugin (VSTState* vstfx)
        return info;
 }
 
-/* A simple 'dummy' audiomaster callback which should be ok,
-   we will only be instantiating the plugin in order to get its info
-*/
-
-static intptr_t
-simple_master_callback (AEffect *, int32_t opcode, int32_t, intptr_t, void *, float)
+/** wrapper around \ref vstfx_parse_vst_state,
+ * iterate over plugins in shell, translate VST-info into ardour VSTState
+ */
+static void
+vstfx_info_from_plugin (const char *dllpath, VSTState* vstfx, vector<VSTInfo *> *infos, enum ARDOUR::PluginType type)
 {
-#if 0
-       static const char* can_do_strings[] = {
-               X_("supportShell"),
-               X_("shellCategory")
-       };
-       static const int can_do_string_count = sizeof (can_do_strings) / sizeof (char*);
-       static int current_loading_id = 0;
-#endif
+       assert(vstfx);
+       VSTInfo *info;
 
-       if (opcode == audioMasterVersion) {
-               return 2;
+       if (!(info = vstfx_parse_vst_state(vstfx))) {
+               return;
        }
-#if 0
-       else if (opcode == audioMasterCanDo) {
-               for (int i = 0; i < can_do_string_count; i++) {
-                       if (! strcmp(can_do_strings[i], (const char*)ptr)) {
-                               return 1;
+
+       infos->push_back(info);
+#if 1 // shell-plugin support
+       /* If this plugin is a Shell and we are not already inside a shell plugin
+        * read the info for all of the plugins contained in this shell.
+        */
+       if (!strncmp (info->Category, "Shell", 5)
+                       && vstfx->handle->plugincnt == 1) {
+               int id;
+               vector< pair<int, string> > ids;
+               AEffect *plugin = vstfx->plugin;
+
+               do {
+                       char name[65] = "Unknown";
+                       id = plugin->dispatcher (plugin, effShellGetNextPlugin, 0, 0, name, 0);
+                       ids.push_back(std::make_pair(id, name));
+               } while ( id != 0 );
+
+               switch(type) {
+#ifdef WINDOWS_VST_SUPPORT
+                       case ARDOUR::Windows_VST: fst_close(vstfx); break;
+#endif
+#ifdef LXVST_SUPPORT
+                       case ARDOUR::LXVST: vstfx_close (vstfx); break;
+#endif
+                       default: assert(0); break;
+               }
+
+               for (vector< pair<int, string> >::iterator x = ids.begin(); x != ids.end(); ++x) {
+                       id = (*x).first;
+                       if (id == 0) continue;
+                       /* recurse vstfx_get_info() */
+
+                       bool ok;
+                       switch (type) {
+#ifdef WINDOWS_VST_SUPPORT
+                               case ARDOUR::Windows_VST:  ok = vstfx_instantiate_and_get_info_fst(dllpath, infos, id); break;
+#endif
+#ifdef LXVST_SUPPORT
+                               case ARDOUR::LXVST:  ok = vstfx_instantiate_and_get_info_lx(dllpath, infos, id); break;
+#endif
+                               default: ok = false;
+                       }
+                       if (ok) {
+                               // One shell (some?, all?) does not report the actual plugin name
+                               // even after the shelled plugin has been instantiated.
+                               // Replace the name of the shell with the real name.
+                               info = infos->back();
+                               free (info->name);
+
+                               if ((*x).second.length() == 0) {
+                                       info->name = strdup("Unknown");
+                               }
+                               else {
+                                       info->name = strdup ((*x).second.c_str());
+                               }
                        }
                }
-               return 0;
-       }
-       else if (opcode == audioMasterCurrentId) {
-               return current_loading_id;
-       }
+       } else {
+               switch(type) {
+#ifdef WINDOWS_VST_SUPPORT
+                       case ARDOUR::Windows_VST: fst_close(vstfx); break;
 #endif
-       else {
-               return 0;
+#ifdef LXVST_SUPPORT
+                       case ARDOUR::LXVST: vstfx_close (vstfx); break;
+#endif
+                       default: assert(0); break;
+               }
        }
+#endif
 }
 
+
+
+/* *** TOP-LEVEL PLUGIN INSTANTIATION FUNCTIONS *** */
+
+#ifdef LXVST_SUPPORT
 static bool
-vstfx_get_info_from_file(char* dllpath, vector<VSTInfo*> *infos)
+vstfx_instantiate_and_get_info_lx (
+               const char* dllpath, vector<VSTInfo*> *infos, int uniqueID)
 {
-       FILE* infofile;
-       bool rv = false;
-       if ((infofile = vstfx_infofile_for_read (dllpath)) != 0) {
-               rv = vstfx_load_info_file(infofile, infos);
-               fclose (infofile);
-               if (!rv) {
-                       PBD::warning << "Cannot get LinuxVST information form " << dllpath << ": info file load failed." << endmsg;
-               }
+       VSTHandle* h;
+       VSTState* vstfx;
+       if (!(h = vstfx_load(dllpath))) {
+               PBD::warning << "Cannot get LinuxVST information from " << dllpath << ": load failed." << endmsg;
+               return false;
        }
-       return rv;
-}
 
-void
-vstfx_free_info (VSTInfo *info)
-{
-       for (int i = 0; i < info->numParams; i++) {
-               free (info->ParamNames[i]);
-               free (info->ParamLabels[i]);
+       vstfx_current_loading_id = uniqueID;
+
+       if (!(vstfx = vstfx_instantiate(h, simple_master_callback, 0))) {
+               vstfx_unload(h);
+               PBD::warning << "Cannot get LinuxVST information from " << dllpath << ": instantiation failed." << endmsg;
+               return false;
        }
 
-       free (info->name);
-       free (info->creator);
-       free (info->Category);
-       free (info->ParamNames);
-       free (info->ParamLabels);
-       free (info);
-}
+       vstfx_current_loading_id = 0;
 
-void
-vstfx_free_info_list (vector<VSTInfo *> *infos)
-{
-       for (vector<VSTInfo *>::iterator i = infos->begin(); i != infos->end(); ++i) {
-               vstfx_free_info(*i);
-       }
-       delete infos;
-}
+       vstfx_info_from_plugin(dllpath, vstfx, infos, ARDOUR::LXVST);
 
-#ifdef LXVST_SUPPORT
-/** Try to get plugin info - first by looking for a .fsi cache of the
-    data, and if that doesn't exist, load the plugin, get its data and
-    then cache it for future ref
-*/
+       vstfx_unload (h);
+       return true;
+}
+#endif
 
-vector<VSTInfo *> *
-vstfx_get_info_lx (char* dllpath)
+#ifdef WINDOWS_VST_SUPPORT
+static bool
+vstfx_instantiate_and_get_info_fst (
+               const char* dllpath, vector<VSTInfo*> *infos, int uniqueID)
 {
-       FILE* infofile;
        VSTHandle* h;
        VSTState* vstfx;
-       vector<VSTInfo*> *infos = new vector<VSTInfo*>;
+       if(!(h = fst_load(dllpath))) {
+               PBD::warning << "Cannot get Windows VST information from " << dllpath << ": load failed." << endmsg;
+               return false;
+       }
 
-       // TODO check blacklist
-       // TODO pre-check file extension ?
+       vstfx_current_loading_id = uniqueID;
 
-       if (vstfx_get_info_from_file(dllpath, infos)) {
-               PBD::info << "using cache for LinuxVST plugin '" << dllpath << "'" << endmsg;
-               return infos;
+       if(!(vstfx = fst_instantiate(h, simple_master_callback, 0))) {
+               fst_unload(&h);
+               vstfx_current_loading_id = 0;
+               PBD::warning << "Cannot get Windows VST information from " << dllpath << ": instantiation failed." << endmsg;
+               return false;
        }
+       vstfx_current_loading_id = 0;
 
-       if (!(h = vstfx_load(dllpath))) {
-               PBD::warning << "Cannot get LinuxVST information from " << dllpath << ": load failed." << endmsg;
-               return infos;
-       }
+       vstfx_info_from_plugin(dllpath, vstfx, infos, ARDOUR::Windows_VST);
 
-       if (!(vstfx = vstfx_instantiate(h, simple_master_callback, 0))) {
-               vstfx_unload(h);
-               PBD::warning << "Cannot get LinuxVST information from " << dllpath << ": instantiation failed." << endmsg;
-               return infos;
-       }
+       return true;
+}
+#endif
 
-       infofile = vstfx_infofile_for_write (dllpath);
 
-       if (!infofile) {
-               vstfx_close(vstfx);
-               vstfx_unload(h);
-               PBD::warning << "Cannot get LinuxVST information from " << dllpath << ": cannot create new FST info file." << endmsg;
-               return 0;
+
+/* *** ERROR LOGGING *** */
+#ifndef VST_SCANNER_APP
+
+static FILE * _errorlog_fd = 0;
+static char * _errorlog_dll = 0;
+
+static void parse_scanner_output (std::string msg, size_t /*len*/)
+{
+       if (!_errorlog_fd && !_errorlog_dll) {
+               PBD::error << "VST scanner: " << msg;
+               return;
        }
 
-       VSTInfo* info = vstfx_info_from_plugin (vstfx);
-       infos->push_back(info); //XXX
+       if (!_errorlog_fd) {
+               if (!(_errorlog_fd = fopen(vstfx_errorfile_path(_errorlog_dll, 0).c_str(), "w"))) {
+                       if (!(_errorlog_fd = fopen(vstfx_errorfile_path(_errorlog_dll, 1).c_str(), "w"))) {
+                               PBD::error << "Cannot create plugin error-log for plugin " << _errorlog_dll;
+                               free(_errorlog_dll);
+                               _errorlog_dll = NULL;
+                       }
+               }
+       }
 
-       vstfx_write_info_file (infofile, infos);
-       fclose (infofile);
+       if (_errorlog_fd) {
+               fprintf (_errorlog_fd, "%s\n", msg.c_str());
+       } else {
+               PBD::error << "VST scanner: " << msg;
+       }
+}
 
-       vstfx_close (vstfx);
-       vstfx_unload (h);
-       return infos;
+static void
+set_error_log (const char* dllpath) {
+       assert(!_errorlog_fd);
+       assert(!_errorlog_dll);
+       _errorlog_dll = strdup(dllpath);
 }
+
+static void
+close_error_log () {
+       if (_errorlog_fd) {
+               fclose(_errorlog_fd);
+               _errorlog_fd = 0;
+       }
+       free(_errorlog_dll);
+       _errorlog_dll = 0;
+}
+
 #endif
 
-#ifdef WINDOWS_VST_SUPPORT
-#include <fst.h>
 
-vector<VSTInfo *> *
-vstfx_get_info_fst (char* dllpath)
+/* *** THE MAIN FUNCTION THAT USES ALL OF THE ABOVE :) *** */
+
+static vector<VSTInfo *> *
+vstfx_get_info (const char* dllpath, enum ARDOUR::PluginType type, enum VSTScanMode mode)
 {
        FILE* infofile;
-       VSTHandle* h;
-       VSTState* vstfx;
        vector<VSTInfo*> *infos = new vector<VSTInfo*>;
 
-       // TODO check blacklist
-       // TODO pre-check file extension ?
+       if (vstfx_check_blacklist(dllpath)) {
+               return infos;
+       }
 
        if (vstfx_get_info_from_file(dllpath, infos)) {
-               PBD::info << "using cache for VST plugin '" << dllpath << "'" << endmsg;
                return infos;
        }
 
-       if(!(h = fst_load(dllpath))) {
-               PBD::warning << "Cannot get VST information from " << dllpath << ": load failed." << endmsg;
+#ifndef VST_SCANNER_APP
+       std::string scanner_bin_path = ARDOUR::PluginManager::scanner_bin_path;
+
+       if (mode == VST_SCAN_CACHE_ONLY) {
+               /* never scan explicitly, use cache only */
+               return infos;
+       }
+       else if (mode == VST_SCAN_USE_APP && scanner_bin_path != "") {
+               /* use external scanner app */
+
+               char **argp= (char**) calloc(3,sizeof(char*));
+               argp[0] = strdup(scanner_bin_path.c_str());
+               argp[1] = strdup(dllpath);
+               argp[2] = 0;
+
+               set_error_log(dllpath);
+               ARDOUR::SystemExec scanner (scanner_bin_path, argp);
+               PBD::ScopedConnectionList cons;
+               scanner.ReadStdout.connect_same_thread (cons, boost::bind (&parse_scanner_output, _1 ,_2));
+               if (scanner.start (2 /* send stderr&stdout via signal */)) {
+                       PBD::error << "Cannot launch VST scanner app '" << scanner_bin_path << "': "<< strerror(errno) << endmsg;
+                       close_error_log();
+                       return infos;
+               } else {
+                       int timeout = PLUGIN_SCAN_TIMEOUT;
+                       bool no_timeout = (timeout <= 0);
+                       ARDOUR::PluginScanTimeout(timeout);
+                       while (scanner.is_running() && (no_timeout || timeout > 0)) {
+                               if (!no_timeout && !ARDOUR::PluginManager::instance().no_timeout()) {
+                                       if (timeout%5 == 0) {
+                                               ARDOUR::PluginScanTimeout(timeout);
+                                       }
+                                       --timeout;
+                               }
+                               ARDOUR::GUIIdle();
+                               Glib::usleep (100000);
+
+                               if (ARDOUR::PluginManager::instance().cancelled()) {
+                                       // remove info file (might be incomplete)
+                                       vstfx_remove_infofile(dllpath);
+                                       // remove temporary blacklist file (scan incomplete)
+                                       vstfx_un_blacklist(dllpath);
+                                       scanner.terminate();
+                                       close_error_log();
+                                       return infos;
+                               }
+                       }
+                       scanner.terminate();
+               }
+               close_error_log();
+               /* re-read index (generated by external scanner) */
+               vstfx_clear_info_list(infos);
+               if (!vstfx_check_blacklist(dllpath)) {
+                       vstfx_get_info_from_file(dllpath, infos);
+               }
                return infos;
        }
+       /* else .. instantiate and check in in ardour process itself */
+#else
+       (void) mode; // unused parameter
+#endif
 
-       if(!(vstfx = fst_instantiate(h, simple_master_callback, 0))) {
-               fst_unload(&h);
-               PBD::warning << "Cannot get VST information from " << dllpath << ": instantiation failed." << endmsg;
+       bool ok;
+       /* blacklist in case instantiation fails */
+       vstfx_blacklist(dllpath);
+
+       switch (type) {
+#ifdef WINDOWS_VST_SUPPORT
+               case ARDOUR::Windows_VST:  ok = vstfx_instantiate_and_get_info_fst(dllpath, infos, 0); break;
+#endif
+#ifdef LXVST_SUPPORT
+               case ARDOUR::LXVST:  ok = vstfx_instantiate_and_get_info_lx(dllpath, infos, 0); break;
+#endif
+               default: ok = false;
+       }
+
+       if (!ok) {
                return infos;
        }
 
-       infofile = vstfx_infofile_for_write (dllpath);
+       /* remove from blacklist */
+       vstfx_un_blacklist(dllpath);
 
+       /* crate cache/whitelist */
+       infofile = vstfx_infofile_for_write (dllpath);
        if (!infofile) {
-               fst_close(vstfx);
-               //fst_unload(&h); // XXX -> fst_close()
-               PBD::warning << "Cannot get VST information from " << dllpath << ": cannot create new FST info file." << endmsg;
-               return 0;
+               PBD::warning << "Cannot cache VST information for " << dllpath << ": cannot create new FST info file." << endmsg;
+               return infos;
+       } else {
+               vstfx_write_info_file (infofile, infos);
+               fclose (infofile);
        }
+       return infos;
+}
 
-       VSTInfo* info = vstfx_info_from_plugin(vstfx);
-       infos->push_back(info); //XXX
 
-       vstfx_write_info_file (infofile, infos);
-       fclose (infofile);
 
-       fst_close(vstfx);
-       //fst_unload(&h); // XXX -> fst_close()
-       return infos;
+/* *** public API *** */
+
+void
+vstfx_free_info_list (vector<VSTInfo *> *infos)
+{
+       for (vector<VSTInfo *>::iterator i = infos->begin(); i != infos->end(); ++i) {
+               vstfx_free_info(*i);
+       }
+       delete infos;
 }
+
+string
+get_personal_vst_blacklist_dir() {
+       string dir = Glib::build_filename (ARDOUR::user_cache_directory(), "fst_blacklist");
+       /* if the directory doesn't exist, try to create it */
+       if (!Glib::file_test (dir, Glib::FILE_TEST_IS_DIR)) {
+               if (g_mkdir (dir.c_str (), 0700)) {
+                       PBD::error << "Cannot create VST blacklist folder '" << dir << "'" << endmsg;
+                       //exit(1);
+               }
+       }
+       return dir;
+}
+
+string
+get_personal_vst_info_cache_dir() {
+       string dir = Glib::build_filename (ARDOUR::user_cache_directory(), "fst_info");
+       /* if the directory doesn't exist, try to create it */
+       if (!Glib::file_test (dir, Glib::FILE_TEST_IS_DIR)) {
+               if (g_mkdir (dir.c_str (), 0700)) {
+                       PBD::error << "Cannot create VST info folder '" << dir << "'" << endmsg;
+                       //exit(1);
+               }
+       }
+       return dir;
+}
+
+#ifdef LXVST_SUPPORT
+vector<VSTInfo *> *
+vstfx_get_info_lx (char* dllpath, enum VSTScanMode mode)
+{
+       return vstfx_get_info(dllpath, ARDOUR::LXVST, mode);
+}
+#endif
+
+#ifdef WINDOWS_VST_SUPPORT
+vector<VSTInfo *> *
+vstfx_get_info_fst (char* dllpath, enum VSTScanMode mode)
+{
+       return vstfx_get_info(dllpath, ARDOUR::Windows_VST, mode);
+}
+#endif
+
+#ifndef VST_SCANNER_APP
+} // namespace
 #endif