Use the correct lock for reading and writing

Author: MatteoPologruto

v2/pkgs/tools.go

@@ -286,8 +286,8 @@ func rename(base string) extract.Renamer {
 }
 
 func (t *Tools) readInstalled() error {
-	t.mutex.Lock()
-	defer t.mutex.Unlock()
+	t.mutex.RLock()
+	defer t.mutex.RUnlock()
 	// read installed.json
 	installedFile, err := utilities.SafeJoin(t.folder, "installed.json")
 	if err != nil {
@@ -301,8 +301,8 @@ func (t *Tools) readInstalled() error {
 }
 
 func (t *Tools) writeInstalled(path string) error {
-	t.mutex.RLock()
-	defer t.mutex.RUnlock()
+	t.mutex.Lock()
+	defer t.mutex.Unlock()
 
 	parts := strings.Split(path, string(filepath.Separator))
 	tool := parts[len(parts)-2]