Fixed #3448

cs3org · Nov 11, 2022 · 57d686c · 57d686c
1 parent aa98c9c
commit 57d686c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/changelog/unreleased/appprovider-panic-fix.md b/changelog/unreleased/appprovider-panic-fix.md
@@ -0,0 +1,5 @@
+Bugfix: Fixes panic in case of empty configuration
+
+Makes sure the config map is allocated prior to setting it
+
+https://github.com/cs3org/reva/pull/3455
diff --git a/internal/grpc/services/appprovider/appprovider.go b/internal/grpc/services/appprovider/appprovider.go
@@ -200,8 +200,13 @@ func (s *service) Register(ss *grpc.Server) {
 func getProvider(c *config) (app.Provider, error) {
 	if f, ok := registry.NewFuncs[c.Driver]; ok {
 		driverConf := c.Drivers[c.Driver]
-		// share the mime_types config entry to the drivers
-		driverConf["mime_types"] = c.MimeTypes
+		if c.MimeTypes != nil {
+			// share the mime_types config entry to the drivers
+			if driverConf == nil {
+				driverConf = make(map[string]interface{})
+			}
+			driverConf["mime_types"] = c.MimeTypes
+		}
 		return f(driverConf)
 	}
 	return nil, errtypes.NotFound("driver not found: " + c.Driver)