2014-07-08 16:30:12

by Fabian Frédérick

[permalink] [raw]

Subject: [PATCH 1/1] ecryptfs: remove unnecessary break after goto

---

Cc: Tyler Hicks <[email protected]>
Cc: [email protected]
Signed-off-by: Fabian Frederick <[email protected]>
---
fs/ecryptfs/keystore.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/fs/ecryptfs/keystore.c b/fs/ecryptfs/keystore.c
index 4725a07..8f0d619 100644
--- a/fs/ecryptfs/keystore.c
+++ b/fs/ecryptfs/keystore.c
@@ -1846,7 +1846,6 @@ int ecryptfs_parse_packet_set(struct ecryptfs_crypt_stat *crypt_stat,
"(Tag 11 not allowed by itself)\n");
rc = -EIO;
goto out_wipe_list;
- break;
default:
ecryptfs_printk(KERN_DEBUG, "No packet at offset [%zd] "
"of the file header; hex value of "
--
1.8.4.5

2014-07-14 16:20:04

by Tyler Hicks

[permalink] [raw]

Subject: Re: [PATCH 1/1] ecryptfs: remove unnecessary break after goto

---

On 2014-07-08 18:30:07, Fabian Frederick wrote:
> Cc: Tyler Hicks <[email protected]>
> Cc: [email protected]
> Signed-off-by: Fabian Frederick <[email protected]>
> ---

Thanks. I'll get this pushed to the eCryptfs -next branch.

Tyler

> fs/ecryptfs/keystore.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/fs/ecryptfs/keystore.c b/fs/ecryptfs/keystore.c
> index 4725a07..8f0d619 100644
> --- a/fs/ecryptfs/keystore.c
> +++ b/fs/ecryptfs/keystore.c
> @@ -1846,7 +1846,6 @@ int ecryptfs_parse_packet_set(struct ecryptfs_crypt_stat *crypt_stat,
> "(Tag 11 not allowed by itself)\n");
> rc = -EIO;
> goto out_wipe_list;
> - break;
> default:
> ecryptfs_printk(KERN_DEBUG, "No packet at offset [%zd] "
> "of the file header; hex value of "
> --
> 1.8.4.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe ecryptfs" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

---

Attachments:

(No filename) (0.98 kB)
signature.asc (819.00 B)
Digital signature Download all attachments