Whamcloud - gitweb
LU-5878 lfs: migrate file to its proper destination 01/12601/6
authorFrank Zago <fzago@cray.com>
Thu, 6 Nov 2014 17:08:30 +0000 (11:08 -0600)
committerOleg Drokin <oleg.drokin@intel.com>
Thu, 4 Dec 2014 02:28:36 +0000 (02:28 +0000)
commit01bc529ccffdb21d33ab6a36ed64fe96ef9ac55b
tree1b488e8d18edf1973ea64f50a74ce3191d90da84
parentc840ee5e1223934299a05798ae64cdbe10a07157
LU-5878 lfs: migrate file to its proper destination

llapi_file_open_param() is supposed to be returning the opened file
descriptor. However, when llapi_search_ost() is called, it returns 1,
which sets rc to 1, which in turn is confused for an error later, and
returned to the caller. So when the copy happen, the destination file
descriptor is 1 (stdout).

Fixed a typo in the function description, and format the parameters
descriptions.

Fixed a bad indentation.

There's no need to test lum before freeing it since at that point is
not NULL (and free will test it anyway).

Change-Id: I16fe26480b880aa818b1bb706b22bfdd6833d69c
Signed-off-by: frank zago <fzago@cray.com>
Reviewed-on: http://review.whamcloud.com/12601
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Jian Yu <jian.yu@intel.com>
Reviewed-by: Henri Doreau <henri.doreau@cea.fr>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
lustre/tests/sanity.sh
lustre/utils/liblustreapi.c