staging: cpia: fix camera file owner in cpia_open()

PATCH New Vasiliy Kulikov 1 participant 1 message Oct 10, 2010

Subsystems (2)

Staging Kernel janitors

lore

Share & tools

lore

Vasiliy Kulikov

author

2010-10-10 17:28 UTC #0 lore raw

PATCH 1/1 staging: cpia: fix camera file owner in cpia_open()

Use effective UID instead of real UID for camera owner.
There is no need to check for pending signals just before successfull return. Exit in case of pending signal also leaved camera in open state.

Signed-off-by Vasiliy Kulikov <[email protected]>

---
Compile tested.

 drivers/staging/cpia/cpia.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

drivers/staging/cpia/cpia.c +1 -5

index 933ae4c..0e740b8 100644
@@ -3184,13 +3184,9 @@ static int cpia_open(struct file *file)
 		goto oops;
 	}
 
-	err = -EINTR;
-	if(signal_pending(current))
-		goto oops;
-
 	/* Set ownership of /proc/cpia/videoX to current user */
 	if(cam->proc_entry)
-		cam->proc_entry->uid = current_uid();
+		cam->proc_entry->uid = current_euid();
 
 	/* set mark for loading first frame uncompressed */
 	cam->first_frame = 1;

Signature · 1 line

1.7.0.4