diff options
| author | Jean Delvare <khali@linux-fr.org> | 2007-09-03 10:51:51 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2007-09-14 12:13:41 -0400 |
| commit | 7b9fbc3e30f785412a26819aa4daf0b6c27f6c53 (patch) | |
| tree | 7ed0e5a77b1e11cf7f5c616dae9a41749eb70d31 /drivers | |
| parent | f5e4feead773cb449212eacfa254a238a953d4d5 (diff) | |
V4L/DVB (6147): Pwc: Fix a broken debug message
Commit 85237f202d46d55c1bffe0c5b1aa3ddc0f1dce4d introduced the
following warning (with CONFIG_USB_PWC_DEBUG=y):
drivers/media/video/pwc/pwc-if.c: In function "pwc_video_close":
drivers/media/video/pwc/pwc-if.c:1211: warning: "i" may be used uninitialized in this function
This is true, and can cause a broken debug message to be logged.
Here's a fix.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/media/video/pwc/pwc-if.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/pwc/pwc-if.c b/drivers/media/video/pwc/pwc-if.c index 3d81966d8c42..931b274bffca 100644 --- a/drivers/media/video/pwc/pwc-if.c +++ b/drivers/media/video/pwc/pwc-if.c | |||
| @@ -1243,7 +1243,7 @@ static int pwc_video_close(struct inode *inode, struct file *file) | |||
| 1243 | PWC_ERROR("Failed to power down camera (%d)\n", i); | 1243 | PWC_ERROR("Failed to power down camera (%d)\n", i); |
| 1244 | } | 1244 | } |
| 1245 | pdev->vopen--; | 1245 | pdev->vopen--; |
| 1246 | PWC_DEBUG_OPEN("<< video_close() vopen=%d\n", i); | 1246 | PWC_DEBUG_OPEN("<< video_close() vopen=%d\n", pdev->vopen); |
| 1247 | } else { | 1247 | } else { |
| 1248 | pwc_cleanup(pdev); | 1248 | pwc_cleanup(pdev); |
| 1249 | /* Free memory (don't set pdev to 0 just yet) */ | 1249 | /* Free memory (don't set pdev to 0 just yet) */ |