Message ID | 20220524225816.6830-5-laurent.pinchart@ideasonboard.com |
---|---|
State | Accepted |
Headers | show |
Series |
|
Related | show |
diff --git a/test/yaml-parser.cpp b/test/yaml-parser.cpp index 959a657478fa..944b564bbe22 100644 --- a/test/yaml-parser.cpp +++ b/test/yaml-parser.cpp @@ -48,10 +48,11 @@ protected: if (fd == -1) return false; - FILE *fh = fdopen(fd, "w"); - fputs(content.c_str(), fh); + int ret = write(fd, content.c_str(), content.size()); + close(fd); - fclose(fh); + if (ret != static_cast<int>(content.size())) + return false; return true; }
There's no point in wrapping a fd into a FILE to then only call fwrite() and fclose(). Use write() and close() directly. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- test/yaml-parser.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)