Allow VSI filesystem access for data files.

[Kurt Schwehr]

Another Googler and I are working on a patch to allow GDAL_DATA to be from a VSI filesystem. For example:

CPLPushFinderLocation( "/vsimem/GDAL_DATA/" );

This will make sandboxing GDAL easier and make systems easier where the GDAL_DATA files have to be written to a filesystem somewhere on startup.

This will be changes like VSIFRead -> VSIFReadL for the main gdal code, but we have not figured out how to properly handle the libgeotiff code. We could use some suggestion for libgeotiff.

I will post some of the patches for discussion before committing anything.

[Kurt Schwehr]

In OGRSpatialReference::SetFromUserInput():

--- ogrspatialreference.cpp     (revision 33380)
+++ ogrspatialreference.cpp     (working copy)
@@ -2064,18 +2064,18 @@
 /* -------------------------------------------------------------------- */
 /*      Try to open it as a file.                                       */
 /* -------------------------------------------------------------------- */
-    FILE        *fp;
+    VSILFILE    *fp;
     const size_t nBufMax = 100000;
     char        *pszBufPtr, *pszBuffer;
     size_t      nBytes;
 
-    fp = VSIFOpen( pszDefinition, "rt" );
+    fp = VSIFOpenL( pszDefinition, "rt" );
     if( fp == NULL )
         return OGRERR_CORRUPT_DATA;
 
     pszBuffer = (char *) CPLMalloc(nBufMax);
-    nBytes = VSIFRead( pszBuffer, 1, nBufMax-1, fp );
-    VSIFClose( fp );
+    nBytes = VSIFReadL( pszBuffer, 1, nBufMax-1, fp );
+    VSIFCloseL( fp );
 
     if( nBytes == nBufMax-1 )
     {

[Kurt Schwehr]

First patch: r33399 for ogrspatialreference.cpp

[Kurt Schwehr]

r33509 and r33508

[Kurt Schwehr]

r33533 and r33534

[Kurt Schwehr]

r33693

[Even Rouault]

Seems this has been addressed. Closing

[Kurt Schwehr]

Not quite there yet. I have a few more changes to come.

[Kurt Schwehr]

r33952 for ogr_srs_pci.cpp

[Even Rouault]

Removing target milestone as it corresponds to a now closed milestone.

[Jukka Rahkonen]

Are there still some changes to come?

[Even Rouault]

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to ​https://github.com/OSGeo/gdal/issues if it is not already reported there.