This repository has been archived by the owner. It is now read-only.

/SikuliX-2014

xdotool parameter should be -v instead of version in LinuxUtil.java - Segmentation fault #253

Closed
pla1 opened this Issue Apr 29, 2017 · 2 comments

Comments

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
2 participants
@pla1 @RaiMan
@pla1

pla1 commented Apr 29, 2017

Hi,

I think CommandLine.parse("xdotool version") should be CommandLine.parse("xdotool -v") in the checkFeatureAvailability method of LinuxUtil.java.

On Ubuntu 14.04 and Ubuntu 16.04 xdotool version fails with a segmentation fault.

It is a bug in xdotool that any unknown parameter that is passed causes a segfault. See my tests below:

htplainf@ub-mguibot:~$ cat /etc/issue
Ubuntu 16.04.2 LTS \n \l
htplainf@ub-mguibot:~$ xdotool -v
xdotool version 3.20150503.1
htplainf@ub-mguibot:~$ xdotool version
Segmentation fault (core dumped)
htplainf@ub-mguibot:~$ xdotool any_unknown_param_causes_segfault
Segmentation fault (core dumped)
htplainf@ub-mguibot:~$

Thank-you!

PLA

@pla1 pla1 changed the title from xdotool parameter should -v instead of version in LinuxUtil.java - Segmentation fault to xdotool parameter should be -v instead of version in LinuxUtil.java - Segmentation fault Apr 29, 2017

@RaiMan

This comment has been minimized.

Show comment
Hide comment
@RaiMan

RaiMan Apr 30, 2017

Owner

Thanks for finding:

made a bug: https://bugs.launchpad.net/sikuli/+bug/1687263

... and will fix it in version 1.1.2
Owner

RaiMan commented Apr 30, 2017

Thanks for finding:

made a bug: https://bugs.launchpad.net/sikuli/+bug/1687263

... and will fix it in version 1.1.2
@RaiMan

This comment has been minimized.

Show comment
Hide comment
@RaiMan

RaiMan Apr 30, 2017

Owner

fixed per today: see
https://raiman.github.io/SikuliX-2014/nightly.html
Owner

RaiMan commented Apr 30, 2017

fixed per today: see
https://raiman.github.io/SikuliX-2014/nightly.html

@RaiMan RaiMan closed this Apr 30, 2017

@toschneck toschneck referenced this issue in ConSol/sakuli May 4, 2017

Open

Selenium extension #247

toschneck added a commit to sakuli/SikuliX-2014 that referenced this issue Aug 23, 2017

@toschneck
* make error output of availability check in Linux Util more readable: … 
…ConSol/sakuli#266, ConSol/sakuli#247

* manual merge RaiMan#253 , commit f9841c4 @RaiMan RaiMan committed on Apr 30
bab5166

toschneck added a commit to sakuli/SikuliX-2014 that referenced this issue Aug 24, 2017

@toschneck
* make error output of availability check in Linux Util more readable: … 
…ConSol/sakuli#266, ConSol/sakuli#247

* manual merge RaiMan#253 , commit f9841c4 @RaiMan RaiMan committed on Apr 30

(cherry picked from commit bab5166)
c12eb94
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.