From: Xu Wang <cngesaint@gmail.com>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, stefanha@gmail.com,
Xu Wang <cngesaint@gmail.com>,
xiawenc@linux.vnet.ibm.com
Subject: [Qemu-devel] [PATCH V2 2/5] Add WIN32 platform support for backing_file_loop_check()
Date: Mon, 8 Jul 2013 03:26:03 -0400 [thread overview]
Message-ID: <1373268366-14508-3-git-send-email-cngesaint@gmail.com> (raw)
In-Reply-To: <1373268366-14508-1-git-send-email-cngesaint@gmail.com>
Signed-off-by: Xu Wang <cngesaint@gmail.com>
---
block.c | 94 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 94 insertions(+)
diff --git a/block.c b/block.c
index 53b1a01..8dc6ded 100644
--- a/block.c
+++ b/block.c
@@ -4431,6 +4431,83 @@ bdrv_acct_done(BlockDriverState *bs, BlockAcctCookie *cookie)
bs->total_time_ns[cookie->type] += get_clock() - cookie->start_time_ns;
}
+#ifdef _WIN32
+static int get_lnk_target_file(const char *lnk_file, char *filepath)
+{
+ unsigned int flag, offet;
+ unsigned int sflag;
+ char uch;
+ int i = 0;
+
+ FILE *fd = fopen(lnk_file, "rb");
+ if (!fd) {
+ error_report("Open file %s failed.", lnk_file);
+ return -1;
+ }
+ fread(&flag, 4, 1, fd);
+ if (flag != 0x4c) {
+ error_report("%s is not a lnk file.", lnk_file);
+ fclose(fd);
+ return -1;
+ }
+ fseek(fd, 0x14, SEEK_SET);
+ fread(&flag, 4, 1, fd);
+ fseek(fd, 0x4c, SEEK_SET);
+
+ if (flag & 0x01) {
+ fread(&sflag, 2, 1, fd);
+ fseek(fd, sflag, SEEK_CUR);
+ }
+
+ offset = ftell(fd);
+ fseek(fd, offset + 0x10, SEEK_SET);
+ fread(&flag, 4, 1, fd);
+ fseek(fd, flag + offset, SEEK_SET);
+
+ do {
+ fread(&uch, 1, 1, fd);
+ filepath[i++] = uch;
+ } while (uch != '\0');
+
+ fclose(fd);
+ return 0;
+}
+
+static long get_win_inode(const char *filename)
+{
+ char pbuf[MAX_PATH_LEN], *p;
+ long inode;
+ struct stat sbuf;
+ char path[MAX_PATH_LEN];
+
+ /* If filename contains .lnk, it's a shortcuts. Target file
+ * * need to be parsed.
+ * */
+ if (strstr(filename, ".lnk")) {
+ if (get_lnk_target_file(filename, path)) {
+ error_report("Parse .lnk file %s failed.", filename);
+ return -1;
+ }
+ } else {
+ memcpy(path, filename, sizeof(filename));
+ }
+
+ if (stat(path, &sbuf) == -1) {
+ error_report("get file %s stat error.", path);
+ return -1;
+ }
+ if (GetFullPathName(path, MAX_PATH_LEN, pbuf, &p) != 0) {
+ inode = 11003;
+ for (p = pbuf; *p != '\0'; p++) {
+ inode = inode * 31 + *(unsigned char *)p;
+ }
+ return (inode * 911) & 0x7FFF;
+ }
+
+ return -1;
+}
+#endif
+
static gboolean str_equal_func(gconstpointer a, gconstpointer b)
{
return strcmp(a, b) == 0;
@@ -4475,7 +4552,15 @@ bool bdrv_backing_file_loop_check(const char *filename, const char *fmt,
error_report("Get file %s stat failed.", filename);
goto err;
}
+ #ifdef _WIN32
+ inode = get_win_inode(filename);
+ if (inode == -1) {
+ error_report("Get file %s inode failed.", filename);
+ goto err;
+ }
+ #else
inode = (long)sbuf.st_ino;
+ #endif
}
filename = backing_file;
@@ -4488,7 +4573,16 @@ bool bdrv_backing_file_loop_check(const char *filename, const char *fmt,
error_report("Get file %s stat failed.", filename);
goto err;
}
+
+ #ifdef _WIN32
+ inode = get_win_inode(filename);
+ if (inode == -1) {
+ error_report("Get file %s inode failed.", filename);
+ goto err;
+ }
+ #else
inode = (long)sbuf.st_ino;
+ #endif
if (g_hash_table_lookup_extended(inodes, &inode, NULL, NULL)) {
error_report("Backing file '%s' creates an infinite loop.",
--
1.8.1.4
next prev parent reply other threads:[~2013-07-08 7:26 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-08 7:26 [Qemu-devel] [PATCH V2 0/5] Add infinite loop check for backing file chain Xu Wang
2013-07-08 7:26 ` [Qemu-devel] [PATCH V2 1/5] Refine and export infinite loop checking in collect_image_info_list() Xu Wang
2013-07-10 10:25 ` Fam Zheng
2013-07-10 10:49 ` Fam Zheng
2013-07-12 2:58 ` Xu Wang
2013-07-08 7:26 ` Xu Wang [this message]
2013-07-10 10:44 ` [Qemu-devel] [PATCH V2 2/5] Add WIN32 platform support for backing_file_loop_check() Fam Zheng
2013-07-15 6:09 ` Xu Wang
2013-07-15 6:30 ` Fam Zheng
2013-07-08 7:26 ` [Qemu-devel] [PATCH V2 3/5] Check infinite loop in bdrv_img_create() Xu Wang
2013-07-10 10:52 ` Fam Zheng
2013-07-08 7:26 ` [Qemu-devel] [PATCH V2 4/5] Add backing file loop check in change_backing_file() Xu Wang
2013-07-10 10:57 ` Fam Zheng
2013-07-08 7:26 ` [Qemu-devel] [PATCH V2 5/5] Add infinite loop check in drive_init() Xu Wang
2013-07-10 11:00 ` Fam Zheng
2013-07-10 11:13 ` [Qemu-devel] [PATCH V2 0/5] Add infinite loop check for backing file chain Fam Zheng
2013-07-10 11:42 ` Kevin Wolf
2013-07-10 17:20 ` Eric Blake
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1373268366-14508-3-git-send-email-cngesaint@gmail.com \
--to=cngesaint@gmail.com \
--cc=kwolf@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
--cc=xiawenc@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).