Fix mode check to only care about owner/setuid.

codebutler · codebutler · commit 7d910232c234 · 2013-12-29T10:10:03.000-08:00
diff --git a/backend/src/unix_platform.hpp b/backend/src/unix_platform.hpp
@@ -68,7 +68,7 @@ class UnixPlatform : public AbstractPlatform
     if ((file_stat.st_mode & S_ISUID) && file_stat.st_uid != 0)
       throw runtime_error("backend is setuid but owner is not root!");
 
-    return (file_stat.st_uid == 0 && file_stat.st_mode == MODE);
+    return (file_stat.st_uid == 0 && (file_stat.st_mode & S_ISUID));
   }
   
   void fix_permissions() {

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ class UnixPlatform : public AbstractPlatform`
`68`	`68`	`if ((file_stat.st_mode & S_ISUID) && file_stat.st_uid != 0)`
`69`	`69`	`throw runtime_error("backend is setuid but owner is not root!");`
`70`	`70`
`71`		`- return (file_stat.st_uid == 0 && file_stat.st_mode == MODE);`
	`71`	`+ return (file_stat.st_uid == 0 && (file_stat.st_mode & S_ISUID));`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`void fix_permissions() {`