[PATCH v2 1/3] Staging: Android: Fixes TODO file

[PATCH v2 1/3] Staging: Android: Fixes TODO file

[Sohny Thomas]

- removed non-existent issue from TODO file
kuid_t or uid_t not present in staging/android

Signed-of-by: Sohny Thomas <sohnythomas@zoho.com>
---
drivers/staging/android/TODO | 7 -------
1 file changed, 7 deletions(-)

diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
index 06954cd..b15fb0d 100644
--- a/drivers/staging/android/TODO
+++ b/drivers/staging/android/TODO
@@ -5,13 +5,6 @@ TODO:
- make sure things build as modules properly
- add proper arch dependencies as needed
- audit userspace interfaces to make sure they are sane
- - kuid_t should never be exposed to user space as it is
- kernel internal type. Data structure for this kuid_t is:
- typedef struct {
- uid_t val;
- } kuid_t;
- - This bug is introduced by Xiong Zhou in the patch bd471258f2e09
- - ("staging: android: logger: use kuid_t instead of uid_t")

Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
Brian Swetland <swetland@google.com>

Re: [PATCH v2 1/3] Staging: Android: Fixes TODO file

[Frans Klaver]

On Wed, Jul 1, 2015 at 3:44 PM, Sohny Thomas <sohnythomas@zoho.com> wrote:
>
> - removed non-existent issue from TODO file
> kuid_t or uid_t not present in staging/android
>
> Signed-of-by: Sohny Thomas <sohnythomas@zoho.com>

s,-of-,-off-,

You can remove the leading dash (-). Could you elaborate on why this
issue is non existent?

Thanks,
Frans

Re: [PATCH v2 1/3] Staging: Android: Fixes TODO file

[Sohny Thomas]

On Wednesday 01 July 2015 07:43 PM, Frans Klaver wrote:
> On Wed, Jul 1, 2015 at 3:44 PM, Sohny Thomas <sohnythomas@zoho.com> wrote:
>>
>> - removed non-existent issue from TODO file
>> kuid_t or uid_t not present in staging/android
>>
>> Signed-of-by: Sohny Thomas <sohnythomas@zoho.com>
> 
> s,-of-,-off-,
> 
> You can remove the leading dash (-). Could you elaborate on why this
> issue is non existent?
Thanks for catching the off. According to the TODO the kuid_t is not present in any files under staging/android
So I said its non-existent


> 
> Thanks,
> Frans
> 

Re: [PATCH v2 1/3] Staging: Android: Fixes TODO file

[Dan Carpenter]

On Wed, Jul 01, 2015 at 07:14:38PM +0530, Sohny Thomas wrote:
> 
> - removed non-existent issue from TODO file
> kuid_t or uid_t not present in staging/android
> 
> Signed-of-by: Sohny Thomas <sohnythomas@zoho.com>


The patch is all mangled.  It barely seems to match the description but
I can't tell for sure because of the mangling.

regards,
dan carpenter

Re: [PATCH v2 1/3] Staging: Android: Fixes TODO file

[Sohny Thomas]

On Wednesday 01 July 2015 07:57 PM, Dan Carpenter wrote:
> On Wed, Jul 01, 2015 at 07:14:38PM +0530, Sohny Thomas wrote:
>>
>> - removed non-existent issue from TODO file
>> kuid_t or uid_t not present in staging/android
>>
>> Signed-of-by: Sohny Thomas <sohnythomas@zoho.com>
> 
> 
> The patch is all mangled.  It barely seems to match the description but
> I can't tell for sure because of the mangling.
I sent an another version, Hope its not mangled now 
Thanks for your time.

-Sohny
> 
> regards,
> dan carpenter
> 

[PATCH v3 ] Staging: Android: Fixes for TODO file

[Sohny Thomas]

Removed non-existent issue from TODO file
No instance of kuid_t or uid_t as mentioned by the file is present in staging/android directory

Signed-off-by: Sohny Thomas <sohnythomas@zoho.com>
---
 drivers/staging/android/TODO | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
index 06954cd..b15fb0d 100644
--- a/drivers/staging/android/TODO
+++ b/drivers/staging/android/TODO
@@ -5,13 +5,6 @@ TODO:
        - make sure things build as modules properly
        - add proper arch dependencies as needed
        - audit userspace interfaces to make sure they are sane
-       - kuid_t should never be exposed to user space as it is
-          kernel internal type. Data structure for this kuid_t is:
-          typedef struct {
-               uid_t val;
-          } kuid_t;
-       - This bug is introduced by Xiong Zhou in the patch bd471258f2e09
-       - ("staging: android: logger: use kuid_t instead of uid_t")

 Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
 Brian Swetland <swetland@google.com>
-- 

Re: [PATCH v3 ] Staging: Android: Fixes for TODO file

[Greg KH]

On Wed, Jul 01, 2015 at 09:19:11PM +0530, Sohny Thomas wrote:
> Removed non-existent issue from TODO file
> No instance of kuid_t or uid_t as mentioned by the file is present in staging/android directory
> 
> Signed-off-by: Sohny Thomas <sohnythomas@zoho.com>
> ---
>  drivers/staging/android/TODO | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
> index 06954cd..b15fb0d 100644
> --- a/drivers/staging/android/TODO
> +++ b/drivers/staging/android/TODO
> @@ -5,13 +5,6 @@ TODO:
>         - make sure things build as modules properly
>         - add proper arch dependencies as needed
>         - audit userspace interfaces to make sure they are sane
> -       - kuid_t should never be exposed to user space as it is
> -          kernel internal type. Data structure for this kuid_t is:
> -          typedef struct {
> -               uid_t val;
> -          } kuid_t;
> -       - This bug is introduced by Xiong Zhou in the patch bd471258f2e09
> -       - ("staging: android: logger: use kuid_t instead of uid_t")
> 
>  Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
>  Brian Swetland <swetland@google.com>

Doesn't apply to the tree :(