ffspart: Increase MAX_LINE to above PATH_MAX

Message ID 20181203001936.8284-1-stewart@linux.ibm.com
State Under Review
Headers show
Series
  • ffspart: Increase MAX_LINE to above PATH_MAX
Related show

Checks

Context Check Description
snowpatch_ozlabs/snowpatch_job_snowpatch-skiboot success Test snowpatch/job/snowpatch-skiboot on branch master
snowpatch_ozlabs/apply_patch success master/apply_patch Successfully applied

Commit Message

Stewart Smith Dec. 3, 2018, 12:19 a.m.
Otherwise we saw failures in CI and the ~221 character paths Jankins
likes to have.

Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
---
 external/ffspart/ffspart.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Samuel Mendoza-Jonas Dec. 11, 2018, 2:16 a.m. | #1
On Mon, 2018-12-03 at 11:19 +1100, Stewart Smith wrote:
> Otherwise we saw failures in CI and the ~221 character paths Jankins
> likes to have.
> 
> Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
> ---
>  external/ffspart/ffspart.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/external/ffspart/ffspart.c b/external/ffspart/ffspart.c
> index 179ed582170b..eeee0d4d657b 100644
> --- a/external/ffspart/ffspart.c
> +++ b/external/ffspart/ffspart.c
> @@ -50,7 +50,7 @@
>   *  Plus \n 40
>   *  Lets do 50.
>   */
> -#define MAX_LINE 255
> +#define MAX_LINE (PATH_MAX+255)
>  #define MAX_TOCS 10
>  #define SEPARATOR ','
>  

Reviewed-by: Samuel Mendoza-Jonas <sam@mendozajonas.com>

Patch

diff --git a/external/ffspart/ffspart.c b/external/ffspart/ffspart.c
index 179ed582170b..eeee0d4d657b 100644
--- a/external/ffspart/ffspart.c
+++ b/external/ffspart/ffspart.c
@@ -50,7 +50,7 @@ 
  *  Plus \n 40
  *  Lets do 50.
  */
-#define MAX_LINE 255
+#define MAX_LINE (PATH_MAX+255)
 #define MAX_TOCS 10
 #define SEPARATOR ','